Pull-Request Schritt 2: Überprüfen
Übersicht
ÜbersichtDieser Artikel erklärt, wie der Überprüfungsprozess für eine Pull Request abgeschlossen wird, an dem sowohl der Ersteller als auch die zugewiesenen Reviewer beteiligt sind. Gründliche Reviews sind entscheidend, um die Codequalität aufrechtzuerhalten.
Best Practices
Fügen Sie immer mindestens einen Reviewer hinzu, um Verantwortlichkeit sicherzustellen.
Verwenden Sie das Beschreibungsfeld, um Prüfern die wichtigsten Änderungen zusammenzufassen.
Mergen Sie nur, wenn alle Konflikte behoben sind und alle Tests bestanden wurden.
Verwenden Sie Änderungen anfordern anstatt einfach nicht zu genehmigen, da dies einen klaren Arbeitsablauf beibehält.
Vermeiden Sie Selbstreviews und das Mergen ohne Reviewer, es sei denn, es handelt sich um einen zeitkritischen Fix.
Pull-Request-Übersicht-Tab
Die Details einer Pull Request helfen Reviewern und Mitwirkenden, Kontext zur Anfrage zu erhalten. Der Übersicht Tab hat zwei Abschnitte zu überprüfen: Pull-Request-Details und Pull-Request-Reviewer.
Pull-Request-Details
Status
Aktueller Zustand der Pull Request (Erstellt, Review angefordert, Änderungen angefordert, und Genehmigt)
Ziel-Branch
Der Branch, in den Änderungen gemerged werden.
Quell-Branch
Der Branch, aus dem die Änderungen stammen.
Erstellt von
Benutzer, der die Pull Request erstellt hat.
Erstellungsdatum
Wann die Pull Request erstmals erstellt wurde.
Aktualisierungsdatum
Wann die letzte Aktivität oder Änderung an der Pull Request stattgefunden hat.
Beschreibung
Zusammenfassung der Änderungen für diese Pull Request. Geschrieben vom Ersteller der Pull Request.
Pull-Request-Reviewer
Die Reviewer Abschnitt zeigt die Benutzer, die zum Review dieser Pull Request zugewiesen sind. Der Status jedes Reviewers wird durch ein Symbol angezeigt, das (stehenden Review, angeforderte Änderungen oder genehmigt) illustriert. Siehe die Anweisungen unten für weitere Details zum Review-Prozess.
Pull Request überprüfen
Auf Merge-Konflikte warten
Wenn eine Pull Request erstellt wird, überprüft sie zunächst auf Merge-Konflikte, angezeigt durch ein graues Banner.
Warten Sie, bis das Banner sich in eines der folgenden ändert:
Grün: Dieser Branch ist mergefähig
Orange: Dieser Branch hat Konflikte, die gelöst werden müssen
Wenn der Branch mergefähig ist, können Sie mit der Überprüfung der Pull Request fortfahren, indem Sie den Anweisungen in diesem Artikel folgen.
Wenn der Branch Konflikte hat, müssen Sie die Konflikte beheben, bevor Sie mit der Überprüfung dieser Pull Request fortfahren. Siehe den folgenden Artikel, um zu lernen, wie Konflikte gelöst werden.
(Optional) Reviewer hinzufügen
Das Hinzufügen von Reviewern sichert die Codequalität, hilft, Konflikte frühzeitig zu erkennen, und erzwingt Teamgenehmigungsprozesse. Alle Reviewer müssen auf Genehmigen klicken, bevor die Pull Request gemerged werden kann.
Selbst-Review
Wenn keine Reviewer hinzugefügt werden, können Sie Ihre eigenen Pull-Request-Änderungen überprüfen. Keine formale Überprüfung oder Genehmigung ist erforderlich. Nach der Überprüfung können Sie ohne zusätzliche Genehmigung mergen.
Im Reviewer Abschnitt, klicken Sie auf das Stift Symbol.
Klicken Sie Hinzufügen für jeden Benutzer, den Sie diese Pull Request überprüfen lassen möchten.
Klicken Sie Speichern.
Die hinzugefügten Reviewer werden im Reviewer Abschnitt aufgeführt, und der Pull-Request-Status ändert sich zu Review angefordert.
Änderungen überprüfen
Um die Pull Request zu überprüfen, muss sich jeder Reviewer bei Cloud DevOps anmelden und die folgenden Schritte ausführen:
Auf Reviews wartende Pull Requests sind mit dem Status Review angefordertmarkiert, und die Reviewer' Symbole werden angezeigt.
Selbst-Review
Wenn Sie eine Selbstprüfung durchführen, müssen Sie trotzdem jede Komponente vor dem Mergen überprüfen.
Klicken Sie Pull Requests aus dem linken Seitenmenü.
Klicken Sie die Pull Request, die Sie überprüfen möchten.
Klicken Sie auf den Geänderte Komponenten Tab.
Klicken Sie auf eine Komponente, um sie zu überprüfen.
Alle Komponenten sollten überprüft werden.
Beim Anzeigen einer Komponente wird der Quellcode für Ziel und Quelle nebeneinander mit hervorgehobenen Unterschieden angezeigt.
Rot: Zu entfernende Zeilen.
Grün: Hinzuzufügende Zeilen.
Wiederholen Sie den Überprüfungsprozess für jede Komponente.
(Optional) Änderungen anfordern
Verwenden Sie beim Überprüfen Änderungen anfordern , um anzuzeigen, dass vor der Genehmigung Korrekturen erforderlich sind. Dieser Status fordert den Ersteller der Pull Request auf, den Quell-Branch mit den notwendigen Änderungen zu aktualisieren. Da die Anforderung nicht angibt, was geändert werden muss, kommunizieren Sie die erforderlichen spezifischen Änderungen direkt an Ihr Team.
Klicken Sie Änderungen anfordern.
Nachdem Sie Änderungen angefordert haben, ändert sich der Pull-Request-Status zu Änderungen angefordert.
Zur Überprüfung erneut einreichen
Nachdem die angeforderten Änderungen vorgenommen wurden, reichen Sie die Pull Request zur erneuten Prüfung bei dem Reviewer ein, der sie angefordert hat.
Klicken Sie die Pull Request.
Klicken Sie auf den Aktualisieren Symbol neben dem Reviewer, der die Überprüfung angefordert hat.
Genehmigen
Sobald ein Reviewer seine Überprüfung aller Komponenten und alle abgeschlossenen Änderungsanforderungen abgeschlossen hat, muss er die Pull Request genehmigen, bevor sie gemerged werden kann.
Die Pull Request wird nicht genehmigt, bis alle Reviewer auf Genehmigen.
Selbst-Review
Wenn Sie selbst prüfen, können Sie nicht Genehmigen die Pull Request. Da es jedoch keine Reviewer gibt, ist vor dem Mergen keine Genehmigung erforderlich.
Klicken Sie Genehmigen.
Nachdem alle Reviewer genehmigt haben, ändert sich der Pull-Request-Status zu Genehmigt, und sie kann gemerged werden.
Nächster Schritt: Mergen
Nachdem die Pull Request genehmigt wurde, sind die Änderungen aus dem Quell-Branch bereit, in den Ziel-Branch gemerged zu werden. Fahren Sie mit dem folgenden Artikel für Anweisungen zum Merge-Prozess fort.
Zuletzt aktualisiert
War das hilfreich?