Attempted Bugfix in GameRuleLogic.kt

[soerendomroes]

Besteht alle Tests aber funktioniert nicht als build

Bug: movementDistance errechnet falsche Distanz für LastMove-Züge, da er den Piranha der gezogen wurde doppelt zählt (Am Anfang (Leeres Feld, weil neues Board -> Fehler) und dann nochmal, wenn er über die anderen Felder iteriert.

@TomekRD

[soerendomroes]

@xeruf

Sehe ich das richtig, dass der workflow Java 8 aufsetzt?

Ich hätte erwwartet, dass wir mittlerweile bei Java 11 sind. Weißt du dazu etwas?

[soerendomroes]

[soerendomroes]

Hm, scheint jetzt wieder zu gehen?! @xeruf ist das normal?

[soerendomroes]

@xeruf @TomekRD Beim starten der GUI bekomme ich folgenden Fehler:

Unable to load class 'sc.gradle.PlaceholderPlugin'.

This is an unexpected error. Please file a bug containing the idea.log file.

@TomekRD hast du den Fehler auch und kannst deshalb nicht testen?
@xeruf der Fehler scheint daher zu kommen, dass wir einen gradle composite build statt buildSrc benutzen. Ich verstehe aber nicht ganz warum "Whilst it’s not really a problem for small projects, big ones with hundreds of modules are affected badly.". Was ist hier die Idee?

[xeruf]

hatte ich noch nie bisher den fehler, aber ich teste diese Woche nochmal

das composite build wieder zu entfernen ist schon länger in überlegung, weil es eigentlich nur ein shortcut sein sollte aber immer mal wieder probleme macht, ist auch in den verschiedenen gradle update PRs typischerweise mit dabei
der workflow arbeitet mit matrix und die enthält aktuell nur java 8, weil die docs task sonst fehlschlägt - auch teil der pending updates

[xeruf]

Bitte für solche behobenen fehler möglichst auch gleich nen test schreiben (TDD) der vorher fehlschlug und dann mit dem fix funktioniert, dann lässt sich das leichter nachvollziehen. Können wir ggf. auch kurz Montag gemeinsam anschauen.

[xeruf]

Unable to load class 'sc.gradle.PlaceholderPlugin'.
This is an unexpected error. Please file a bug containing the idea.log file.

der fehler kommt in intellij? mal ./gradlew run ausprobiert?

[soerendomroes]

Guter Tipp, da gibt es bei mir einen anderen Fehler, kannst du das reproduzieren @xeruf ?

* What went wrong:
Could not determine the dependencies of task ':run'.
> Could not resolve all task dependencies for configuration ':runtimeClasspath'.
   > Could not find com.github.edvin:tornadofx2:21e933fd41.
     Searched in the following locations:
       - https://repo.maven.apache.org/maven2/com/github/edvin/tornadofx2/21e933fd41/tornadofx2-21e933fd41.pom
       - https://maven.wso2.org/nexus/content/groups/wso2-public/com/github/edvin/tornadofx2/21e933fd41/tornadofx2-21e933fd41.pom
       - https://jitpack.io/com/github/edvin/tornadofx2/21e933fd41/tornadofx2-21e933fd41.pom
     Required by:
         project :

[soerendomroes]

Wenn das früher unter https://oss.sonatype.org/content/repositories/snapshots war, dann ist das jetzt weg. Sonatype hat das ganze publishing und snapshot Zeug umgestellt und das Repo ist seit Jahren nicht geupdated.