Wie richte ich einen Code-Review mit Gitlab ein?

Question

Wie erstellt man einen Code-Review mit Gitlab? Ich sehe es als eine Funktion auf der Gitlab-Website aufgeführt, aber ich kann nicht scheinen, Anweisungen zu finden, wie man einen aufstellt (für jeden Fall würde irgendein Link zu einem Gitlab Benutzerhandbuch am meisten geschätzt werden).

Einige meiner Suche hat angezeigt, dass "Merge Requests" sind der Weg zu gehen ... aber ich finde sie begrenzend. Eine ausgegebene Zusammenführungsanforderung zeigt alle Commits zwischen einer Verzweigung und der anderen an. Ich scheine nur Diffs zu sehen, 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 Änderungen über 10 Commits in einem Dev-Zweig eingereicht. Wenn ich eine Zusammenführungsanforderung für diesen Dev-Zweig von der Integration ausgabe, sehe ich 10 Commits, von denen jede die inkrementellen Änderungen anzeigt, die an der Datei vorgenommen wurden ... Ich möchte die gesamte Sache überprüfen. Es ist neu!

Banne ich hier den falschen Baum? Gibt es ein aktuelles Code-Review-Tool, das ich in GitLab verwenden kann, oder sind Merge-Requests der Weg zu gehen, und wenn sie sind, verwende ich sie falsch? Was ist der beste Weg, um hier einen korrekten Code-Review zu erstellen?

Answer 1

Sie können den übermittelten Code in der Merge-Anforderung für ein anderes Repository oder im aktuellen Repository sehen.
Beispiel http://demo.gitlab.com/diaspora/diaspora/commits/master

Dann können Sie Kommentare zu engagierten Dateiänderungen (Button Antworten) oder die

Die resultierenden communication is code reviewhttp://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

Beispiel begehen ganzen hinzuzufügen. Ich empfehle jedoch, Code-Review auf einem PC mit Face-to-Face-Kommunikation wann immer möglich zu machen, und Tools für die Aufzeichnung von Ergebnissen oder wenn mehr Formalität benötigt wird.

Für eine Dateirevue, die viele Commits, z. http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md betrachten Sie es mit Schuld zu verstehen, wer was getan hat. In dieser Ansicht gibt es jedoch keine Möglichkeit zum Kommunizieren und Hinzufügen von Kommentaren. Ich würde empfehlen, nur Änderungen als Kommentare in diesem Fall hinzuzufügen.

Answer 2

Ja. Zusammenführungsanfragen sind, wie Peer-Reviews durchgeführt werden.

Es sollte eine Registerkarte 'diff' geben, die die Änderungen aller Commits zeigt (hier erwähnt: http://youtu.be/DyAX8ws5OIc?t=3m2s).

Das Video erklärt auch schön, wie es Peer Review verwendet werden kann.

Answer 3

Ich habe Code-Reviews in Gitlab seit über zwei Monaten mit fast keiner Reibung gemacht. Ich habe rss2email eingerichtet, um E-Mail-Benachrichtigungen jedes Mal zu senden, wenn ein Entwickler neue Commits überträgt. Dann benutze ich die Kommentarfunktion von Gitlab für Commits, um einige Kommentare über den Push-Code zu machen.

Leider erlaubt Gitlab 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, was ich bei einem früheren Commit verpasst habe, benutze ich das "blame" -Tool, um das Commit zu finden, das den Code-Abschnitt eingeführt/geändert hat, um kommentiert zu werden.

Es ist weit von perfekt, aber es funktioniert so gut.

Answer 4

Hinweis: da GitLab 6.4, side-by-side diff view verfügbar ist: 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 the comments die Per line comments are now supported (09/2014):

Ihre Teammitglieder auf der Zusammenführung Anfrage im Allgemeinen oder auf bestimmte Zeilen mit Zeilenkommentaren kommentieren.

Das kann immer noch für Code-Review-Aktivität helfen.