Nachdem Sie eine Pull-Anforderung geöffnet haben, geben Prüfer Feedback, das Ihnen hilft, Ihren Code zu verbessern, bevor sie zusammengeführt wird. Das Lösen dieses Feedbacks bedeutet, jeden Kommentar zu verstehen, die von ihr geforderten Änderungen vorzunehmen und sicherzustellen, dass Sie alles behoben haben.
Grundlegendes zu Rezensionsfeedback
Prüfer können allgemeines Feedback hinterlassen, bestimmte Zeilen kommentieren und genaue Änderungen als Vorschläge vorschlagen. Rezensionen werden in der Zeitachse der Pull-Anforderung angezeigt, damit Sie der Diskussion folgen und sehen können, welche Kommentare noch eine Antwort benötigen. Je nach Repositoryeinstellungen kann ein Prüfer auch Ihre Pullanforderung genehmigen oder Änderungen anfordern, die Sie vor dem Zusammenführen adressieren sollten.
Bevor Sie etwas ändern, lesen Sie jeden Kommentar, um seine Absicht zu verstehen. Ein Kommentar weist möglicherweise auf einen Fehler hin, stellen Sie eine Frage, fordern Sie einen anderen Ansatz an, oder schlagen Sie eine Bearbeitung vor, die Sie direkt annehmen können.
Wenn Sie Zugriff auf Copilot haben, kann es Ihnen helfen, Review-Kommentare zu interpretieren und Korrekturen vorzuschlagen, was besonders nützlich ist, wenn ein Pull Request viele Kommentare enthält, die bearbeitet werden müssen.
Implementieren von Änderungen und Beheben von Code
Wenn Sie Ihr Feedback ansprechen, können Sie:
- Wenden Sie eine von einem Reviewer vorgeschlagene Änderung direkt an, wodurch der Vorschlag als Commit auf Ihrem Branch gespeichert wird.
- Nehmen Sie lokal umfangreichere Änderungen vor und pushen Sie neue Commits in den Branch.
Das meiste Feedback wird behoben, indem Ihr Code aktualisiert und neue Commits an den Pullanforderungszweig übertragen werden. Wenn Sie Feedback adressieren, markieren Sie Unterhaltungen als gelöst, damit Sowohl Sie als auch Ihre Prüfer nachverfolgen können, was erledigt wird und was weiterhin Aufmerksamkeit benötigt.
Wenn die erforderlichen Prüfer ihre Genehmigung erteilt haben und keine angeforderten Änderungen mehr ausstehen, kann Ihre Pull Request zusammengeführt werden. Da die Pullanforderung die Verzweigung nachverfolgt, aktualisiert jeder neue Commit die Pullanforderung und führt alle automatisierten Prüfungen erneut aus.
Effizientes Arbeiten durch Feedback
Bei größeren oder mehr überprüften Pullanforderungen helfen Ihnen einige Methoden, Feedback schnell zu behandeln:
- Vorschläge stapelweise annehmen, indem vorgeschlagene Änderungen einem Batch hinzugefügt werden, sodass mehrere angenommene Änderungen in einem einzigen Commit statt in jeweils einem Commit pro Vorschlag landen.
- Lösen Sie alle Zusammenführungskonflikte proaktiv, damit Sie schnell zusammenführen können, sobald die Überprüfung der Pull-Anforderung aufgelöst wurde.
- Fordern Sie Rezensionen erneut an, nachdem Sie wesentliche Änderungen vorgenommen haben, sodass Prüfer wissen, dass die Pull-Anforderung für ein anderes Aussehen bereit ist.
- Erforderliche Überprüfungen verstehen. Wenn ein Branch Genehmigungen oder eine Freigabe durch den Code-Owner erfordert, können eine Review mit „Änderungen angefordert“ oder eine widerrufene Freigabe das Zusammenführen blockieren, bis das Problem behoben ist.
- Verfolgen Sie Feedback außerhalb des Umfangs, indem Sie ein Issue erstellen, das auf den Kommentar verweist, anstatt den Pull Request zu erweitern.
Tools zur Behebung von Rezensionen
Sie müssen Ihr Feedback nicht allein manuell durchgehen:
-
Sehen Sie sich den Pull Request lokal an oder öffnen Sie ihn in GitHub Codespaces, um Probleme zu reproduzieren und Korrekturen zu testen, bevor Sie pushen.
-
Beheben Sie Sicherheitsbefunde, die während der Überprüfung festgestellt wurden. Codeüberprüfungswarnungen für Ihre Änderungen werden in der Pullanforderung angezeigt, damit Sie sie vor dem Zusammenführen beheben können.
-
Verwenden Copilot um Feedback zu interpretieren, Fragen zu beantworten, Korrekturen vorzuschlagen und Zusammenführungskonflikte mit Agenten zu lösen.