Was ist ein Pull Request?
Ein Pull Request ist die Bitte, die Commits eines Branches in einen anderen zu übernehmen, meist nach main. Andere Personen lesen die Änderung, kommentieren sie und geben sie frei, bevor sie zusammengeführt wird.
In einem Team ändert niemand die gemeinsame Hauptlinie allein. Ein zweites Paar Augen findet Fehler, die man selbst übersieht, und das ganze Team lernt, was sich ändert.
Am Ende dieser Lektion
Notiz
Ich kann eine Änderung für ein Review vorbereiten und beurteilen, ob sie in den Hauptbranch darf.
Schlüsselbegriffe
| Begriff | Was er bedeutet |
|---|---|
| Pull Request | die Bitte, einen Branch in einen anderen zu übernehmen |
| Review | das Lesen und Beurteilen einer Änderung durch andere |
| Diff | die Liste dessen, was sich zwischen zwei Ständen ändert |
| Freigabe | die Zustimmung der prüfenden Person |
| Kleine Änderung | ein Vorschlag mit einem Zweck, den man schnell liest |
Die Zellen, die Lernende ausfüllen, bleiben in dieser Vorschau leer.
Ein Pull Request verbindet Branch und Gespräch
Sie arbeiten auf einem eigenen Branch und senden ihn zum Remote. Auf der Webseite des Servers, hier Gitea, eröffnen Sie dann einen Pull Request: von Ihrem Branch nach main.
Die Seite zeigt den Diff und lässt andere Zeilen kommentieren. Haben die Prüfenden die Änderung durchgesehen, geben sie sie frei. Erst danach wird zusammengeführt.
Das Repository nach pruefung klonen
git clone http://gitea:3000/student/hello-git.git pruefungHier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Ihren Namen für diesen Ordner setzen
git -C pruefung config user.name "Studierende"Hier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Ihre Adresse für diesen Ordner setzen
git -C pruefung config user.email "[email protected]"Hier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Einen Branch für den Vorschlag anlegen
git -C pruefung switch -c vorschlag-gaestebuchHier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Eine neue Datei für den Vorschlag schreiben
printf 'Eintraege der Gaeste\n' > pruefung/gaestebuch.txtHier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Die Datei vormerken
git -C pruefung add gaestebuch.txtHier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Den Vorschlag als einen Commit speichern
git -C pruefung commit -m "Gaestebuch angelegt"Hier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Prüfende lesen zuerst, was sich ändert
Ein Review beginnt mit der Frage: Welche Dateien sind betroffen, und welche Commits gehören dazu? Der Diff gegen main beantwortet die erste, das Log zwischen den Branches die zweite.
Pushen Sie den Übungsordner nicht. Sie lesen nur, was Prüfende lesen würden.
Die geänderten Dateien gegen main ansehen
git -C pruefung --no-pager diff --name-status main..vorschlag-gaestebuchHier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Die Commits des Vorschlags auflisten
git -C pruefung --no-pager log --format=%s main..vorschlag-gaestebuchHier angezeigt, nicht ausgeführt. Erstellen Sie ein kostenloses Konto, um „Ausführen“ zu drücken und ihn im Terminal Ihres eigenen Labs einzugeben.
Eine kleine Änderung besteht ein Review leichter
| Merkmal | Gute Änderung | Schwierige Änderung |
|---|---|---|
| Titel | Ein kurzer Satz, der den Zweck nennt | Leer oder "Änderungen" |
| Umfang | Wenige hundert Zeilen mit einem Zweck | Tausende Zeilen mit vielen Zwecken |
| Beschreibung | Sagt, was sich ändert und warum | Fehlt |
| Tests | Neuer Code kommt mit einem Test | Neuer Code ohne Test |
| Geheimnisse | Keine Passwörter in den Dateien | Eine .env-Datei im Commit |
Die Zellen, die Lernende ausfüllen, bleiben in dieser Vorschau leer.
Gute Kommentare nennen Ort, Grund und Vorschlag
Ein Kommentar im Review zeigt auf eine Zeile, sagt, was auffällt, und schlägt etwas vor. "Das ist falsch" hilft wenig. "Hier steht 3 statt 2, die Karte hat nur zwei Preise" hilft.
Nach der Durchsicht gibt es zwei Wege: die Freigabe oder die Bitte um Änderungen. Wer Änderungen verlangt, sagt genau, welche.
Ein häufiger Irrtum
Warnung
Viele glauben, ein Review prüfe die Person. Es prüft die Änderung. Kritik gilt dem Text und dem Verhalten, nicht dem Menschen, und die Freigabe heißt nur: Die Änderung darf in die Hauptlinie.