Ich bin nur neugierig, was der Unterschied zwischen Add a Single Commentund Start a Reviewist, wenn Sie Feedback auf der changed filesRegisterkarte in einer pull request.
Wenn ich zum Beispiel einen kleinen Teil des Codes umgestalte, um ihn zu bereinigen, ist es ein commentoder ein code review?
Danke
Das ist eine gute Funktion in GitHub. Wenn Sie einen einzelnen Kommentar hinzufügen, werden die Prüfer der PR sofort benachrichtigt. Ich benutze das nicht. Ich beginne eine Überprüfung, mache alle meine Kommentare und beende meine Überprüfung. Dadurch werden die Rezensenten sofort über alle meine Kommentare informiert.
Wenn Sie auch nur ein einziges Zeichen im Code ändern, ist dies weder ein Kommentar noch eine Codeüberprüfung; es ist ein weiteres Commit und hat mit einem anderen PR zu tun.
Hier gibt es insbesondere keine richtige Antwort; Meistens hängt es davon ab, wie Ihr Team der Verwendung der GitHub-PRs zustimmt.
Wenn Sie „Eine Überprüfung starten“, müssen Sie, wie Nezih erwähnt hat, nicht nur alle Kommentare speichern und den Benutzer einmal benachrichtigen, sondern auch „Genehmigen“, „Änderungen anfordern“ oder „Kommentar“ auswählen. Wenn Sie ein angeforderter Bewerter waren, wird angezeigt, dass Sie eine Bewertung eingereicht haben (unabhängig davon, was Sie auswählen). Einen einzigen Kommentar zu hinterlassen reicht nicht.
Weil es alles spart, die Person nur einmal zu benachrichtigen, wenn man fertig ist, ist es mir schon mal passiert, dass sich der Code am Ende meiner Rezension schon wieder geändert hat, weil der Autor auf andere Änderungswünsche reagiert hat. Das ist verwirrend und nervig und veranlasst mich manchmal dazu, stattdessen einzelne Kommentare zu hinterlassen. Auf der anderen Seite, wenn ich viele Kommentare abgeben möchte , ist es meiner Meinung nach freundlicher für den Autor, sie im Modus „Überprüfung starten“ zu bündeln. Dies gilt insbesondere dann, wenn sich meine Kommentare aufeinander beziehen.
Wenn Sie einen umgestalteten Code als Vorschlag posten , können Sie beide Optionen verwenden. Wenn Sie der Meinung sind, dass die PR ohne Ihren Vorschlag nicht zusammengeführt werden sollte, wählen Sie „Überprüfung starten“ und „Änderungen anfordern“.
Wenn Ihr Team zugestimmt hat, dass Teamkollegen den Code des anderen für solche Dinge modifizieren können (was ungewöhnlich ist, aber es gibt Situationen, in denen es gut funktioniert), dann würden Sie den Branch auschecken und Ihr eigenes Commit dorthin pushen. Aber vergewissern Sie sich zuerst, dass Sie die Teamnormen verstehen: In meinem Team tun wir das nicht, ohne vorher den Autor zu fragen, um eine mögliche Überfrachtung laufender Arbeiten zu vermeiden.