Feature/ci #358
Feature/ci #358
Conversation
pstoeckle
commented
Sep 26, 2025
pstoeckle
commented
- first draft
peggimann
left a comment
peggimann
left a comment
There was a problem hiding this comment.
Ich denke als Einführung ist dies durchaus sinnvoll! Ich denke aber auch, dass wir noch etwas genauer beschreiben können wie eine CI funktioniert, oder zumindest den Code Snipped der Pipeline erklären, was ist der Trigger, wieso brauchen wir den etc.
Vielen Dank für den coolen Draft!
| ### Aufgabe 1: Erstelle ein GitHub Repository | ||
|
|
||
| 1. Gehe zu [GitHub](https://github.com/new) und erstelle ein neues Repository. | ||
| 2. Nenne das Repository `ci-example` und setze es auf "Public". |
There was a problem hiding this comment.
Gibt es einen bestimmten Grund, wieso das Repo auf Public sein muss? (Ist so oder so default) vielleicht ist es auch eine Option, das Repo private zu lassen und die Accounts der Ausbildner hinzuzufügen?
There was a problem hiding this comment.
Ich hatte im Hinterkopf, dass es da Beschränkungen für die GitHub Actions gibt, wenn du ein Repo auf private hast. Finde dazu aber auf die Schnelle nichts. Von daher kann das auch weg.
There was a problem hiding this comment.
@andiaeg hat mich mal gebeten die IT-Ninja-Warriors Org aufzusetzen. Denke wir könnten die Repositories dort erstellen, dann hätten alle Praxisbildner direkt access.
@andiaeg was meinst du?
There was a problem hiding this comment.
@andiaeg was meinst du?
Soll ich das ändern? Also, dass die Lernenden die Repos in der IT-Ninja-Warriors Org aufsetzen sollen?
There was a problem hiding this comment.
Höi zäme. Ich denke wir sollten das am MI zusammen besprechen. Grundsätzlich denke ich, dass wir auf Github eine Organisation aufsetzen sollten für die Übungen. Vielleicht nennen wir sie aber IT-Ninjas-YYYY (YYYY für das Ausbildungsjahr). Grund sind die Limitierungen. Wir haben nur 500MB zur Verfügung für die Packages und ich bin mir nicht sicher, wie weit das reicht... Weitere Limitierungen: https://github.com/organizations/it-ninjas/billing/plans. In der Zwischenzeit könnte man ja ein POC machen, ob und es mit der IT-Ninja-Warriors Org machbar ist (Repo öffnen für Patrick, Zugriffsrechte für alle PB, Code Hinzufügen, Actions definieren, main-Branch schützen, ...). Passt das?
Alles klar, dann füge ich das noch hinzu |
|
@peggimann So besser? |
Siehe 6e93f4e |
| ### Aufgabe 2a: Teste Deine CI! (Optional) | ||
|
|
||
| 1. Füge dem Java Code einen Fehler hinzu, der verhindert, dass das Java Programm | ||
| kompiliert werden kann oder die Tests fehlschlagen lässt. | ||
| 2. Committe und pushe die Änderungen. | ||
| 3. Gehe zu deinem GitHub Repository und klicke auf den Reiter `Actions`. | ||
| Warte, bis dein Workflow abgeschlossen ist. | ||
| Der Workflow sollte fehlschlagen und mit einem ❌ markiert sein. | ||
| 4. Klicke auf den Workflow, um die Details zu sehen. | ||
| 5. Klicke auf den Job `java-test` und dann auf den Step `Run JUnit tests using Maven`. | ||
| Du solltest die Ausgabe von Maven sehen und dort auf den Fehler | ||
| hingewiesen werden, den du gerade eingeführt hast. | ||
| 6. Fixe den Fehler wieder. | ||
| 7. Committe und pushe die Änderungen. | ||
| 8. Der Workflow sollte wieder erfolgreich sein und mit einem ✅ markiert sein. |
There was a problem hiding this comment.
Vielleicht ist es hier noch eine Überlegung wert die Aufgabe 2a nicht optional zu machen? Die Lernenden sollten an diesem Punkt ja bereits Unit-Tests kennen und fixen können.
Stattdessen würde die Pipeline zuerst fehlschlagen, was direkt zeigt wieso CI wichtig ist. Dann müssten sie die Pipeline fixen, bis sie grün wird.
There was a problem hiding this comment.
Vielleicht ist es hier noch eine Überlegung wert die Aufgabe 2a nicht optional zu machen?
Umgesetzt in 8212c0d
There was a problem hiding this comment.
Stattdessen würde die Pipeline zuerst fehlschlagen, was direkt zeigt wieso CI wichtig ist. Dann müssten sie die Pipeline fixen, bis sie grün wird.
@MasterEvarior würde ich nicht machen. Wenn die Lernenden nicht genau lesen bzw. die Schritte nicht richtig machen, dann schlägt die Pipeline auch fehl, aber die Lernenden denken, dass alles in Ordnung ist. Und die Lernenden merken den Fehler erst, wenn sie es fixen sollen.
Ja sehr cool vielen Dank!!!! |
chrira
left a comment
chrira
left a comment
There was a problem hiding this comment.
Kurz und knackig. Finde es gut, danke.
|
Hoi, |
@pstoeckle Probiers nochmals, ich hab dich hinzugefügt. Evtl. musst du noch eine Einladung annehmen irgendwo. |
Ja! jetzt geht es wieder! hab die kommentare addressiert |
5 participants
