Conversation
3d70429 to
c3503ba
Compare
Migrate maven to maven-publish
soerendomroes
left a comment
There was a problem hiding this comment.
Ich verstehe nicht ganz, warum jetzt alle plugins verschoben werden.
Hattest du getestet, ob man mit dem SimpleClient für die Schüler auch mit einer niedrigeren Java Version auskommt?
| "a player joined" should { | ||
| players[0].joinGame(TestPlugin.TEST_PLUGIN_UUID) | ||
| "create a room for it" { | ||
| await("Room opened") { lobby.games.size shouldBe 1 } | ||
| val room = lobby.games.single() | ||
| room.clients shouldHaveSize 1 | ||
| "return GameResult on step" { | ||
| withClue("return GameResult on step") { | ||
| val roomListener = observeRoom(room.id) | ||
| admin.control(room.id).step(true) | ||
| val result = roomListener.waitForMessage(GameResult::class) | ||
| playerHandlers[0].gameResult shouldBe result | ||
| result.win shouldBe Team.ONE | ||
| result.isRegular shouldBe false | ||
| result.win?.reason.shouldBeInstanceOf<Violation.LEFT>() | ||
| result.win?.winner shouldBe Team.TWO | ||
| result.isRegular shouldBe true | ||
| admin.closed shouldBe false | ||
| } |
There was a problem hiding this comment.
Der Test scheint sich sehr geändert zu haben. Hat sich hier auch das Verhalten geändert?
| log/ | ||
| replays/ No newline at end of file | ||
| replays/ | ||
| plans/ |
There was a problem hiding this comment.
das ist für agentic coding
|
@xeruf Gibt es dazu schon einen GUI branch der damit kompatibel ist? |
|
Der build failed bei mir noch: Muss ich außer zu pullen noch etwas einstellen? |
There was a problem hiding this comment.
2020 war nicht Blokus dran. Das ist Hive. Kannst du das bitte beim Verschieben mit umbenennen?
| @@ -108,6 +108,7 @@ class GameState @JvmOverloads constructor( | |||
| return result | |||
| } | |||
|
|
|||
| /** @suppress */ | |||
There was a problem hiding this comment.
irgendwelches linting I guess
There was a problem hiding this comment.
ach ja das lässt es aus der Dokumentation raus
damit es für die zukunft leichter zu maintainen bleibt wenn wir rotieren
JDK 17 ist das minimum mit Gradle 9, dann müssten die Schüler die gradle version downgraden oder das ohne gradle machen
vorher mal |
|
Leider läuft der Spieler nicht und es scheint auch noch keinen gui-branch geben, um alles zu testen. Das liegt wohl daran, dass der player:run die gui auch baut? |
|
Wenn wir mit für die Schulen noch Java 8 und 11 kompatibel sein wollen, dann müsste man den Spieler wahrscheinlich in ein eigenes Repo mit eigenem build bauen. Wie siehst du das? |
|
ne man könnte das auch händisch mit bundlen - aber wollen wir diese versionen wirklich weiter maintainen? gabs mal ne umfrage bei den tutoren ob die schulen die die versionen noch benutzen wirklich nix anderes haben?
versteh ich nicht, der wird wie üblich mit
ne hat nix mit der gui zu tun |
Die Umfrage dazu gibt es erst zum Ende der Saison. Die Tutoren, mit denen ich sprechen konnte, meinten, dass es noch ein paar Schulen gibt die das brauchen. Und die Versionen sind ja mindestens bis nächstes Jahr noch LTS.
Ok, anscheinend ist mein Developer setup ein bisschen anders als sonst und die Dinge die ich testen möchte. |
|
Leider baut das bei mir (auch nach clean) nicht: Klappt das bei dir einfach? |
.github/workflows/gradle.yml
Outdated
| os: [ubuntu-latest, windows-latest, macos-latest] | ||
| jdk: [11, 24] |
There was a problem hiding this comment.
Hi, kannst du bitte die höchste LTS version nehmen (Java 21)?
There was a problem hiding this comment.
ich dachte ich nehm halt die höchste die supported sein sollte, aber klar geht auch
|
weil das nur mündlich besprochen wurde und hier noch nicht steht - wir gehen runter auf gradle 8, was jdk 11 bis 24 supported und damit alles wichtige abdeckt |
|
https://en.wikipedia.org/wiki/Java_version_history Java 24 ist nicht das neueste Java mit einer kleineren Nummer als 25. Die LTS Versionen werden weiter geupdated, Java 24 ist nicht mehr supported. Es lohnt sich entsprechend beim Test sowas wie 8, 11, 17 und 21 zu testen. Und wir hatten uns auf Gradle 8.14.X geeinigt, da dies auch noch Java 8 supported. |
|
wir nutzen shadow 9.1 weswegen das backend bauen mindestens jdk 11 braucht, aber teilnehmer können trotzdem noch jdk 8 nutzen weil ich shadow da auf 8.3 gesetzt hab - das ist kein problem |
backend itself needs at least jdk 11 to build with shadow 9.1 but this enables participants to still use jdk 8 https://github.com/GradleUp/shadow?tab=readme-ov-file#compatibility-matrix
|
hab chore/gradle7 gui branch nochmal aktualisiert |
supports higher JDK versions, fancy dokka documentation and a lot more improvements including folder structure