Wie richte ich eine Codeüberprüfung mit Gitlab ein?

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?

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):

Ihre Teammitglieder können die Zusammenführungsanforderung im Allgemeinen oder in bestimmten Zeilen mit Zeilenkommentaren kommentieren.

Das kann immer noch bei der Codeüberprüfung hilfreich sein.

6 Jahre später für GitLab 13.1 (Juni 2020) :

Merge Request Reviews wurden in Core verschoben

Merge Request Reviews wurde ursprünglich in GitLab 11.4 als GitLab Premium-Funktion eingeführt und ermöglicht es Merge Request Reviewers:

• mehrere Kommentare gleichzeitig einreichen,
• Reduzierung des Benachrichtigungsrauschens für den Autor der Zusammenführungsanforderung und
• Dies ermöglicht einen kohärenteren und optimierten Überprüfungsprozess.

Seit seiner Einführung haben wir seinen Platz in unserem käuferbasierten Preismodell neu bewertet. Als Teil von 13.1 freuen wir uns, Ihnen mitteilen zu können, dass diese Funktion jetzt auf GitLab Core umgestellt wurde.

Siehe Dokumentation und Ausgabe

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.

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.

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.

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

git co -b FIRST_COMMIT eb67f06c2b3222c0219214b176c41922bc454881
vi README.md
git add README.md
git ci -m "First commit modified so can get full diff against it"
git push --set-upstream origin FIRST-COMMIT

Überprüfen Sie das letzte Commit, nehmen Sie eine Änderung vor, schreiben Sie es fest und drücken Sie es

git co -b master
vi README.md
git add README.md
git ci -m "Last commit modified so can get full diff against it"
git push --set-upstream origin LAST-COMMIT

Erstellen Sie in GitLab / GitHub eine Pull-Anfrage

• Es ist eine Zusammenführung von LAST_COMMIT zu FIRST_COMMIT

Funktioniert bei mir!