Code Review Workflow + Funktionsverzweigung in TFS

Question

Wir beginnen mit Feature-Verzweigungen zu arbeiten, und wir möchten eine Check-in-Richtlinie einrichten, die nur Check-Ins auf die Baseline erlaubt, wenn sie eine Code-Überprüfung haben.

Der neue Code-Review-Workflow im Jahr 2012 ist ziemlich nett, da Sie einfach mit dem Entwickler und anderen Überprüfern interagieren und Codezeilen direkt kommentieren können. Dennoch scheint es, wie MS nicht den Anwendungsfall voll dachte, weil wir einfach in das folgende Problem auftreten:

1. Der Entwickler auf dem Funktionszweig Einchecken in Werke/Regale und zukunfts Integration regelmäßig.

2. Wenn sie die Funktion integrieren möchte, geht sie zurück in die Baseline und fordert eine Überprüfung dieser ausstehenden Änderungen an.

3. Der Reviewer macht mehrere Kommentare und jetzt muss sie einen Code ändern. Wo macht sie das?

Option 1: zum Zweig Gehen Sie zurück, den Code bearbeiten und Check-in den Veränderungen in der Branche. Machen Sie die ausstehenden Änderungen der ersten Zusammenführung rückgängig. Zusammenführen und erneut eine Überprüfung anfordern. Wiederholen Sie den Vorgang, bis keine weiteren Kommentare vorhanden sind. Check-in die Zusammenführung. Das ist nicht so schön, weil alle Review-Kommentare in den ausstehenden Änderungen der Zusammenführung sind, und sie muss an der Verzweigung arbeiten, wo sie die Kommentare nicht direkt sieht.

Option 2: Nehmen Sie die Änderungen direkt an den ausstehenden Änderungen der Zusammenführung vor. Fordern Sie eine Rezension erneut an. Wiederholen Sie den Vorgang, bis keine weiteren Kommentare vorhanden sind. Check-in die Zusammenführung. Wenn sie weiter in der Branche arbeiten möchte, müsste sie eine Vorwärtsintegration vornehmen, weil die Änderungen aus der Überprüfung nicht da sind.

Wie auch immer, die zweite Überprüfung ist immer sehr ärgerlich, weil Sie die Änderungen zwischen der ersten und der zweiten Überprüfung nicht sehen können, da Sie sich immer mit der Basislinie auseinandersetzen.

Fehle ich hier etwas? Gibt es eine weitere Option, mit der die Änderungen einer Überprüfung überprüft werden können? Hat jemand eine bessere Möglichkeit zur Verzweigung und Überprüfung von Code?

. Neu: Mit VS und TFS2013 noch keine Verbesserungen :(

Answer 1

Sie sind nicht etwas fehlen Dies ist ein unglückliches Problem mit der Art und Weise Code Reviews wurden umgesetzt, assoziiert ist, kann sie nur mit einem verbunden werden changeset, nicht zu einer Reihe von Änderungen

Wenn Ihr Team eine hohe Anzahl an Checkins in ihren Feature-Branches gewohnt ist, kann es sehr aufwendig sein, jedes einzelne Changeset mit diesem Tool überprüfen zu lassen Empfehlung

Es gibt einen Trick, es ist nicht ideal, aber es kann helfen. Sie können alle Dateien, die seit dem letzten Einchecken geändert wurden, auschecken (in Ihrem Funktionszweig). Dann fordern Sie eine Überprüfung an. Es erstellt ein Shelveset mit Ihren Änderungen und verknüpft es mit der Überprüfung. Auf diese Weise müssen Sie die Zusammenführung nicht durchführen, bevor Sie die Überprüfung anfordern. Stellen Sie sicher, dass Sie die neueste Version von main in Ihren Feature-Zweig einbinden, bevor Sie diesen Trick ausführen.Dies hat zwei wesentliche Nachteile:

1. Während alle geänderten Dateien mit der Überprüfung verknüpft sind, werden die Änderungen seit der letzten Überprüfung nicht automatisch hervorgehoben. Der Prüfer müsste manuell einen "Vergleich zur Version" durchführen und das Vergleichsziel auswählen.
2. Es gibt eine Grenze von 4000 (von oben) Dateien, die mit einer Überprüfung verknüpft werden können, so dass die Anzahl der Dateien, die Sie als Gruppe überprüfen können, begrenzt ist (ich hoffe, dass Sie 4000 nicht ändern) + Dateien zwischen Integrationen in main).