Freigeben über

Abrufen von Feedback mit Pull Requests

  • Artikel

Pull-Anforderungen unterstützen das Überprüfen und Zusammenführen von Code in einem einzigen kollaborativen Prozess. Sobald ein Entwickler eine Funktion hinzufügt oder einen Fehler behebt, erstellt er eine Pull-Anforderung, um den Prozess der Zusammenführung der Änderungen in den Upstream-Zweig einzuleiten. Die anderen Teammitglieder erhalten dann die Möglichkeit, den Code zu überprüfen und zu genehmigen, bevor er fertiggestellt wird. Verwenden Sie Pull Requests zum Überprüfen von laufenden Arbeiten und für frühzeitiges Feedback zu Änderungen. Aber es gibt keine Verpflichtung, die Änderungen zusammenzuführen. Ein Besitzer kann eine Pull-Anforderung jederzeit aufgeben.

Code überprüfen lassen

Die Überprüfung des Codes im Rahmen einer Pull-Anforderung dient nicht nur dazu, offensichtliche Fehler zu finden; dafür sind Tests da. Ein guter Code-Review deckt weniger offensichtliche Probleme auf, die später zu kostspieligen Problemen führen könnten.

Code-Reviews schützen das Team vor schlechten Zusammenführungen und fehlerhaften Builds, die die Produktivität des Teams beeinträchtigen. Reviews fangen Probleme vor der Zusammenführung auf und schützen wichtige Zweige vor unerwünschten Änderungen.

Code-Reviews fördern und stärken auch die Zusammenarbeit und Kommunikation zwischen Entwicklern. Und das Team erhält klare Verlaufsdaten zu allen Änderungen, die zwischen dem Hauptzweig und den Funktionszweigen vorgenommen wurden.

Tauschen Sie Fachwissen aus und verbreiten Sie Problemlösungsstrategien, indem Sie ein breites Spektrum von Reviewern bei Code-Reviews einsetzen. Die Verbreitung von Fähigkeiten und Wissen macht das Team stärker und widerstandsfähiger.

Geben Sie großartiges Feedback

Qualitativ hochwertige Reviews beginnen mit qualitativ hochwertigem Feedback. Folgende Aspekte spielen für großartiges Feedback in einem Pull Request eine Schlüsselrolle:

  • Bitten Sie die richtigen Personen, den Pull Request zu überprüfen.
  • Stellen Sie sicher, dass die Reviewer wissen, was der Code macht.
  • Geben Sie umsetzbares, positives Feedback.
  • Beantworten Sie Kommentare zeitnah.

Wenn Sie einer Anforderung Gutachter zuweisen, achten Sie darauf, dass Sie die richtige Gruppe von Gutachtern auswählen. Die Prüfer sollten wissen, wie der Code funktioniert, aber auch Entwickler einbeziehen, die in anderen Bereichen arbeiten, damit sie ihre Ideen einbringen können.

Geben Sie eine klare Beschreibung der Änderungen und stellen Sie einen Build des Codes zur Verfügung, in dem die Korrektur oder Funktion funktioniert. Die Gutachter sollten sich bemühen, auch Änderungen, mit denen sie nicht einverstanden sind, zu kommentieren. Identifizieren Sie das Problem und machen Sie konkrete Vorschläge, was anders gemacht werden könnte. Dieses Feedback hat eine klare Absicht und ist für den Besitzer des Pull Requests leicht verständlich.

Der Besitzer der Pull-Anforderung sollte auf Kommentare antworten, Vorschläge annehmen oder erklären, warum er sie ablehnt. Einige Vorschläge sind gut, liegen aber möglicherweise außerhalb des Rahmens der Anforderung. Nutzen Sie diese Vorschläge, und erstellen Sie neue Arbeitselemente und Featurebranches getrennt vom Pull Request, um diese Änderungen vorzunehmen.

Schützen von Branches mit Richtlinien

Es gibt ein paar kritische Zweige in einem Repo, bei denen sich Teams darauf verlassen, dass sie immer in gutem Zustand sind, wie z. B. der main Zweig. Teams können mit Plattformen wie GitHub und Azure DevOps Pull-Anforderungen verlangen, um Änderungen an diesen Zweigen vorzunehmen. Die Pushvorgänge von Entwicklern, die Änderungen direkt an diese geschützten Branches pushen, werden zurückgewiesen.

Fügen Sie zusätzliche Bedingungen zu Pull-Anforderungen hinzu, um ein höheres Maß an Codequalität in wichtigen Zweigen zu erzwingen. Ein sauberer Build des zusammengeführten Codes und die Genehmigung durch mehrere Prüfer sind einige zusätzliche Anforderungen, die häufig zum Schutz wichtiger Zweige eingesetzt werden.

Weitere Informationen

GitHub hat eine ausführliche Dokumentation darüber, wie Sie mit Pull-Anforderungen Änderungen an Ihrer Arbeit vorschlagen können.

Lesen Sie mehr darüber, wie Sie bei Code-Reviews gutes Feedback geben können und wie Sie Vorlagen für Pull-Anforderungen verwenden, um Ihren Gutachtern Richtlinien an die Hand zu geben. Azure DevOps bietet auch eine umfangreiche Erfahrung mit Pull-Anforderungen, die einfach zu nutzen und je nach Bedarf skalierbar ist.