Conversation
ProfWider
left a comment
ProfWider
left a comment
There was a problem hiding this comment.
Die Commit-Struktur ist schon etwas abenteuerlich, aber die Tests an sich erfüllen die Anforderungen. Leider ist einer Ihrer Tests noch rot. Vermutlich haben Sie ihn mit einem Fix für einen anderen Test wieder kaputt gemacht (in Zusammenhang mit der führenden Null). Wichtig ist, dass am Ende alle Tests in Kombination mit allen Bugfixes grün werden, d.h. gradle test, welches alle Tests ausführt, muss insgesamt grün werden. Sie haben noch bis morgen Abend um einen Fix nachzureichen, so dass alles grün wird (besser lokal checken bevor Sie pushen).
ProfWider
left a comment
ProfWider
left a comment
There was a problem hiding this comment.
Bestanden! 👍
Commits sind natürlich ganz schön durcheinander mittlerweile aber ok.
Wenn Sie Lust haben, können Sie ja jetzt ganz ohne Druck nochmal neu forken, clonen, dann Stück für Stück die Änderugen rüber ziehen und dabei direkt die Commits erzeugen, und dann nochmal einen sauberen PR stellen. Da Sie die richtige Lösung ja jetzt schon haben, sollte das sogar recht schnell gehen.
2 participants
No description provided.