Wie richtet man eine Codeüberprüfung mit Gitlab ein? Ich sehe es als eine Funktion auf der Gitlab-Website aufgelistet, aber ich kann anscheinend keine Anweisungen zum Einrichten finden (im Übrigen wäre jeder Link zu einem Gitlab-Benutzerhandbuch sehr willkommen).
Einige meiner Suchanfragen haben ergeben, dass "Zusammenführungsanforderungen" der richtige Weg sind ... aber ich finde sie einschränkend. Eine ausgegebene Zusammenführungsanforderung zeigt alle Commits zwischen einem Zweig und dem anderen an. Ich kann anscheinend nur Unterschiede anzeigen, die für jedes einzelne Commit generiert wurden. Angenommen, ich habe eine Datei, die ich überprüfen möchte. Es ist eine neue Datei, aber ich habe über 10 Commits in einem Entwicklungszweig Änderungen daran übermittelt. Wenn ich eine Zusammenführungsanforderung für diesen Entwicklungszweig aus der Integration heraus gebe, werden 10 Commits angezeigt, von denen jedes die inkrementellen Änderungen an der Datei anzeigt. Ich möchte das Ganze überprüfen. Es ist neu!
Belle ich hier den falschen Baum an? Gibt es ein aktuelles Tool zur Codeüberprüfung, das ich in GitLab verwenden kann, oder sind Zusammenführungsanforderungen der richtige Weg, und wenn ja, verwende ich sie falsch? Was ist der beste Weg, um hier eine ordnungsgemäße Codeüberprüfung einzurichten?
quelle
Antworten:
Hinweis: Seit GitLab 6.4 ist eine Side-by-Side-Diff-Ansicht verfügbar: siehe " Pull Request 5308 ".
(Juli 2013)
Es gibt jedoch noch keine Möglichkeit, jede Zeile zu kommentieren, nur auf Dateiebene.Daniel Sokolowski erwähnt in den Kommentaren, dass Kommentare pro Zeile jetzt unterstützt werden (09/2014):
Das kann immer noch bei der Codeüberprüfung hilfreich sein.
6 Jahre später für GitLab 13.1 (Juni 2020) :
Siehe Dokumentation und Ausgabe
quelle
Ich mache seit über zwei Monaten Code-Reviews in Gitlab, fast ohne Reibung. Ich habe rss2email so eingerichtet, dass E-Mail-Benachrichtigungen jedes Mal gesendet werden , wenn ein Entwickler neue Commits sendet. Dann verwende ich die Kommentarfunktion von Gitlab für Commits, um einige Kommentare zum Push-Code abzugeben.
Leider erlaubt Gitlab keine Kommentare zu Dateien selbst, nur in Commits (genau wie Github, denke ich). Immer wenn ich mich in einer Situation befinde, in der ich etwas kommentieren muss, das ich in einem vorherigen Commit verpasst habe, verwende ich das Schuld-Tool, um das Commit zu finden, das den zu kommentierenden Codeabschnitt eingeführt / geändert hat.
Es ist alles andere als perfekt, aber es funktioniert bisher gut.
quelle
Sie können den übermittelten Code in der Zusammenführungsanforderung für ein anderes Repository oder im aktuellen Repository anzeigen.
Beispiel http://demo.gitlab.com/diaspora/diaspora/commits/master
Anschließend können Sie Kommentare zu festgeschriebenen Dateiänderungen (Schaltfläche Reply) oder zum gesamten Festschreiben hinzufügen
Beispiel http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c
Die resultierende Kommunikation ist eine Codeüberprüfung . Ich persönlich empfehle jedoch, nach Möglichkeit eine Codeüberprüfung auf einem PC mit persönlicher Kommunikation durchzuführen und Tools zum Aufzeichnen von Ergebnissen zu verwenden oder wenn mehr Formalität erforderlich ist.
Für eine Datei-Revue mit vielen Commits, z. B. http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md, schauen Sie sich diese an, um blamezu verstehen, wer was getan hat. In dieser Ansicht gibt es jedoch keine Möglichkeit, zu kommunizieren und Kommentare hinzuzufügen. Ich würde empfehlen, in diesem Fall nur Änderungen als Kommentare hinzuzufügen.
quelle
Ja. Durch Zusammenführungsanfragen werden Peer Reviews durchgeführt.
Es sollte eine Registerkarte "diff" geben, auf der die Änderungen aller Commits angezeigt werden (hier erwähnt: http://youtu.be/DyAX8ws5OIc?t=3m2s ).
Das Video erklärt auch gut, wie es für Peer Review verwendet werden kann.
quelle
Der normale Anwendungsfall von Codeüberprüfungen besteht darin, Code in einem Zweig vor dem Zusammenführen mit Master oder ähnlichem zu überprüfen. Ich habe eine Situation, in der ich ein Projekt entwickelt habe und möchte, dass der gesamte Code von allen im Team überprüft wird.
Was ich getan habe war:
Überprüfen Sie das erste Commit, nehmen Sie eine Änderung vor, schreiben Sie es fest und drücken Sie es
Überprüfen Sie das letzte Commit, nehmen Sie eine Änderung vor, schreiben Sie es fest und drücken Sie es
Erstellen Sie in GitLab / GitHub eine Pull-Anfrage
Funktioniert bei mir!
quelle
git br --delete --force origin FIRST_COMMIT LAST_COMMIT; git br --delete --force FIRST_COMMIT LAST_COMMIT
). Sie können einen anderen Verzweigungsmaster verwenden, um Änderungen daran zu enthalten, oder separate Probleme manuell erstellen. Erstellen Sie später einen oder mehrere Zweige (z. B. einen pro Ausgabe), wenn zu viele Rückmeldungen vorliegen.