[D2G] Spotless in der CI

[AMatutat]

• Spotless auf den Task Ordner im Studi-Reo ausführen
• Gradle Task erstellen

[AHeinisch]

Wenn ich darüber nachdenke, um spotless laufen zu bekommen, müssen wir es bei den Studis ausführen und nicht in der Vorgabe.
Um sicherzustellen, dass kein Betrug gemacht wird könnte man für Spotless eine kleine Java Datei mit ungültiger Formatierung in den Ordner kopieren und wenn diese nicht formatiert wurde handelt es sich um eine defekte Konfiguration bei den Studis.

[cagix]

Wenn ich darüber nachdenke, um spotless laufen zu bekommen, müssen wir es bei den Studis ausführen und nicht in der Vorgabe.
Um sicherzustellen, dass kein Betrug gemacht wird könnte man für Spotless eine kleine Java Datei mit ungültiger Formatierung in den Ordner kopieren und wenn diese nicht formatiert wurde handelt es sich um eine defekte Konfiguration bei den Studis.

wow ... das wäre ganz schöner aufwand.

dann könnte man ja auch hingehen und das studi-repo in einen unterordner vom vorgabe-repo klonen und dann aus den vorgaben die skripte starten. das wäre einfacher.

aber ich hatte das so verstanden, dass @AMatutat einen weg gefunden hatte?

[tgrothe]

dann könnte man ja auch hingehen und das studi-repo in einen unterordner vom vorgabe-repo klonen und dann aus den vorgaben die skripte starten. das wäre einfacher.

So hab ich das eigentlich auch gedacht. Die Studis clonen sich das Vorgabe-Repo (zurzeit vorgabe-dummy) und clonen darin dann ihr Studi-Repo... Dann wäre die "top level" build.gradle doch vorhanden, so dass einfach spotlessApply aufgerufen werden kann.

Auf "Serverseite", also dann später im (Studi-)PR/CI, kann die Formatierung dann noch einmal überprüft werden (mit spotlessCheck).

Oder hab ich nun einen Denkfehler gerade?

[cagix]

dann könnte man ja auch hingehen und das studi-repo in einen unterordner vom vorgabe-repo klonen und dann aus den vorgaben die skripte starten. das wäre einfacher.

So hab ich das eigentlich auch gedacht. Die Studis clonen sich das Vorgabe-Repo (zurzeit vorgabe-dummy) und clonen darin dann ihr Studi-Repo... Dann wäre die "top level" build.gradle doch vorhanden, so dass einfach spotlessApply aufgerufen werden kann.

Auf "Serverseite", also dann später im (Studi-)PR/CI, kann die Formatierung dann noch einmal überprüft werden (mit spotlessCheck).

Oder hab ich nun einen Denkfehler gerade?

vielleicht.

1. wir haben ein zentrales repo mit zwei top-level-ordnern loesung und vorgabe.
2. der inhalt des ordners vorgabe wird der inhalt des studi-repos (über den zwischenschritt git subtree und forken des damit bestückten repos).
3. im ordner vorgabe soll eine gradle-config sein, mit der die studis arbeiten und die qualität ihrer lösung checken können. das läuft komplett im studi-repo (bzw. im ordner vorgabe, wenn wir das aus sicht unseres repos betrachten).
4. in der ci bzw. auf meinem rechner werden die studi-repos als ordner ausgecheckt und unser repo daneben gelegt. dann wird immer der lösungsordner aus dem jeweiligen studi-repo und der vorgabe-ordner für den task aus unserem repo gemeinsam als sourcen betrachtet und kompiliert und bewertet. das muss die konfiguration herstellen.
5. mir ist grad nicht klar, wie @AMatutat das konkret gemacht hatte: man kann die studi-repos "neben" den ordner mit unserem repo auschecken, man könnte sie aber auch "in" dem ordner mit unserem repo auschecken (dann liegen die studi-ordner parallel zu unseren ordnern loesung und vorgabe - das erscheint mir grad einfacher, weil wir dann einfach toplevel eine übergeordnete gradle-konfig in unserem repo anlegen könnten). allerdings kommt dann git evtl. durcheinander, und @AMatutat sucht ja nach commits in den studi-repos. aber es könnte gehen ...

[tgrothe]

@cagix Ich finde den Kommentar gerade nicht wieder, wo es darum ging, ob Spotless bei nicht formatierten Dateien weiterlaufen könnte, aber ich habe dieses Issue dazu gefunden:

diffplug/spotless#287

[cagix]

@cagix Ich finde den Kommentar gerade nicht wieder, wo es darum ging, ob Spotless bei nicht formatierten Dateien weiterlaufen könnte, aber ich habe dieses Issue dazu gefunden:

diffplug/spotless#287

jetzt hast du mich verloren. wie höngt dein kommentar mit den darüber zusammen?

[tgrothe]

@cagix Ich finde den Kommentar gerade nicht wieder, wo es darum ging, ob Spotless bei nicht formatierten Dateien weiterlaufen könnte, aber ich habe dieses Issue dazu gefunden:
diffplug/spotless#287

jetzt hast du mich verloren. wie höngt dein kommentar mit den darüber zusammen?

Es gibt nur einen Zusammenhang zu Spotless, nicht direkt zu deinem Kommentar...

[cagix]

ah 😀

[AKirsch1]

Ersetzt mit #38.