Screenshots eingefügt
parent
2af8c408fe
commit
4e15ab45d7
@ -53,16 +53,21 @@ Ein Pull Request ist deine formale Anfrage, deine Änderungen in `main` zu über
|
|||||||
|
|
||||||
1. Gehe in Gitea zu unserem Projekt-Repository.
|
1. Gehe in Gitea zu unserem Projekt-Repository.
|
||||||
2. Klicke auf den Reiter **"Pull Requests"**.
|
2. Klicke auf den Reiter **"Pull Requests"**.
|
||||||
3. Klicke auf den blauen Knopf **"Neuer Pull Request"**.
|

|
||||||
|
3. Klicke auf den blauen Knopf **"Neuer Pull-Request"**.
|
||||||
|

|
||||||
4. **Wähle die Branches aus:**
|
4. **Wähle die Branches aus:**
|
||||||
* **Basis-Branch (links):** Wähle `main` (dort soll es hin).
|
* **Basis-Branch (links):** Wähle `main` (dort soll es hin).
|
||||||
* **Vergleichs-Branch (rechts):** Wähle deinen Branch (z.B. `feature/dein-feature-name`).
|
* **Vergleichs-Branch (rechts):** Wähle deinen Branch (z.B. `feature/dein-feature-name`).
|
||||||
5. Klicke erneut auf **"Pull Request erstellen"**.
|
5. Klicke erneut auf **"Neuer Pull-Request"**.
|
||||||
|

|
||||||
6. Fülle die Maske aus:
|
6. Fülle die Maske aus:
|
||||||
* **Titel:** Ein klarer Titel (z.B. "Feature: User-Login hinzugefügt").
|
* **Titel:** Ein klarer Titel (z.B. "Feature: User-Login hinzugefügt").
|
||||||
* **Beschreibung:** Beschreibe kurz, *was* du getan hast und *warum*.
|
* **Beschreibung:** Beschreibe kurz, *was* du getan hast und *warum*.
|
||||||
* **Reviewer: (Optional)** Du kannst den Pull Request einem oder mehreren Kollegen direkt zuweisen. Wenn du niemanden auswählst, geht der Pull Request in den allgemeinen Pool und wartet auf eine proaktive Prüfung durch das Team.
|
* **Reviewer: (Optional)** Du kannst den Pull Request einem oder mehreren Kollegen direkt zuweisen. Wenn du niemanden auswählst, geht der Pull Request in den allgemeinen Pool und wartet auf eine proaktive Prüfung durch das Team.
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
### Schritt 4: Auf Feedback reagieren
|
### Schritt 4: Auf Feedback reagieren
|
||||||
Dein Reviewer (oder ein Kollege, der den PR proaktiv prüft) wird sich den Code nun ansehen.
|
Dein Reviewer (oder ein Kollege, der den PR proaktiv prüft) wird sich den Code nun ansehen.
|
||||||
|
|
||||||
@ -122,15 +127,19 @@ Es gibt zwei Wege, wie ein Pull Request zu dir kommt:
|
|||||||
* **Dies ist entscheidend, damit keine Pull Requests übersehen werden.**
|
* **Dies ist entscheidend, damit keine Pull Requests übersehen werden.**
|
||||||
* Weise dir den Pull Request selbst zu (falls gewünscht) oder beginne direkt mit Schritt 2.
|
* Weise dir den Pull Request selbst zu (falls gewünscht) oder beginne direkt mit Schritt 2.
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
### Schritt 2: Den Code prüfen
|
### Schritt 2: Den Code prüfen
|
||||||
Öffne den Pull Request in Gitea. Konzentriere dich auf diese Bereiche:
|
Öffne den Pull Request in Gitea. Konzentriere dich auf diese Bereiche:
|
||||||
|

|
||||||
1. **Beschreibung:** Verstehst du, *was* der Autor tun wollte? Ist das Ziel klar?
|
1. **Beschreibung:** Verstehst du, *was* der Autor tun wollte? Ist das Ziel klar?
|
||||||
2. **Reiter "Geänderte Dateien":** Das ist der Kern deiner Arbeit. Gehe die Änderungen durch.
|
2. **Reiter "Geänderte Dateien":** Das ist der Kern deiner Arbeit. Gehe die Änderungen durch.
|
||||||
* **Funktioniert das?** Löst der Code das beschriebene Problem?
|
* **Funktioniert das?** Löst der Code das beschriebene Problem?
|
||||||
* **Gibt es offensichtliche Fehler?** (z.B. fehlende Fehlerbehandlung).
|
* **Gibt es offensichtliche Fehler?** (z.B. fehlende Fehlerbehandlung).
|
||||||
* **Ist der Code verständlich?** Sind Variablen und Funktionen sinnvoll benannt?
|
* **Ist der Code verständlich?** Sind Variablen und Funktionen sinnvoll benannt?
|
||||||
* **Passt es zum Rest des Projekts?** Hält sich der Code an unsere Architektur?
|
* **Passt es zum Rest des Projekts?** Hält sich der Code an unsere Architektur?
|
||||||
|
|
||||||
|

|
||||||
|
|
||||||
### Schritt 3: Feedback in Gitea geben
|
### Schritt 3: Feedback in Gitea geben
|
||||||
Während du die Dateien prüfst, kannst du direkt Kommentare hinterlassen:
|
Während du die Dateien prüfst, kannst du direkt Kommentare hinterlassen:
|
||||||
@ -139,6 +148,8 @@ Während du die Dateien prüfst, kannst du direkt Kommentare hinterlassen:
|
|||||||
* Schreibe dein Feedback.
|
* Schreibe dein Feedback.
|
||||||
* Wenn du alle Kommentare gesammelt hast, klicke oben auf **"Review abschicken"** (oder "Submit Review").
|
* Wenn du alle Kommentare gesammelt hast, klicke oben auf **"Review abschicken"** (oder "Submit Review").
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
### Schritt 4: Den Review abschließen
|
### Schritt 4: Den Review abschließen
|
||||||
Wenn du deinen Review abschickst, musst du eine Entscheidung treffen:
|
Wenn du deinen Review abschickst, musst du eine Entscheidung treffen:
|
||||||
|
|
||||||
@ -150,4 +161,5 @@ Wenn du deinen Review abschickst, musst du eine Entscheidung treffen:
|
|||||||
* **Wann:** Der Code ist fehlerfrei, verständlich und bereit für den `main`-Branch.
|
* **Wann:** Der Code ist fehlerfrei, verständlich und bereit für den `main`-Branch.
|
||||||
* **Was passiert:** Du klickst auf "Approve". Der Pull Request wird als "genehmigt" markiert.
|
* **Was passiert:** Du klickst auf "Approve". Der Pull Request wird als "genehmigt" markiert.
|
||||||
|
|
||||||
|

|
||||||
**Wichtig:** Der Autor des Pull Requests ist (sofern nicht anders vereinbart) anschließend dafür verantwortlich, den Merge durchzuführen, nachdem die Genehmigung erteilt wurde.
|
**Wichtig:** Der Autor des Pull Requests ist (sofern nicht anders vereinbart) anschließend dafür verantwortlich, den Merge durchzuführen, nachdem die Genehmigung erteilt wurde.
|
||||||
Loading…
Reference in New Issue
Block a user