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/ |
|
@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?
| return result | ||
| } | ||
|
|
||
| /** @suppress */ |
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? |
| 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)?
supports higher JDK versions, fancy dokka documentation and a lot more improvements including folder structure