it-swarm.com.de

TFS Code Reviews - Zeigt aktualisierte Dateien als Antwort auf Kommentare an

Wir fangen an, die in die Vorschau von VS 2012 und VS 2013 integrierte Codeüberprüfungsfunktion zu verwenden. Das Anfordern der Überprüfung und das Hinzufügen von Kommentaren scheinen ziemlich einfach zu sein. Wenn jemand Kommentare hinzufügt, in denen die Änderung des Codes angefordert wird, wie nimmt der Anforderer diese Änderungen vor und zeigt sie an?

Der Prozess würde also so ablaufen:

  1. Person 1 fordert eine Codeüberprüfung an.
  2. Person 2 fügt Kommentare hinzu und wählt "Needs Work".
  3. Person 1 nimmt die notwendigen Änderungen vor.

Wie zeigt Person 1 diese Änderungen nun für Person 2 an? Sie können Kommentare hinzufügen und senden, aber die Dateien ändern sich nicht. Ich gehe davon aus, dass die Dateien aus dem Änderungssatz stammen, der erstellt wurde, als die ursprüngliche Überprüfung angefordert wurde. Sollte Person 1 diese Überprüfung schließen und eine zweite Überprüfung anfordern?

Das Erstellen einer zweiten Überprüfung scheint nicht optimal zu sein, da Sie den Verlauf Ihrer Konversation darüber verlieren, warum Sie die Änderungen vorgenommen haben.

Es gibt unzählige Websites, auf denen gezeigt wird, wie die grundlegenden Funktionen der Codeüberprüfungen verwendet werden. Gibt es jedoch Websites, auf denen die besten Vorgehensweisen aufgeführt sind? Wer sollte zum Beispiel die Kontrollkästchen neben Dateien aktivieren?

118
John

Der Prozess würde also so ablaufen:

  1. Person 1 fordert eine Codeüberprüfung an.
  2. Person 2 fügt Kommentare hinzu und wählt "Needs Work".
  3. Person 1 nimmt die notwendigen Änderungen vor.
  4. Person 1 Aktualisiert das mit der Codeüberprüfung verknüpfte Regal
  5. Person 1 fügt Kommentare hinzu, um die Diskussion fortzusetzen
  6. Wiederholen Sie die Schritte 2 bis 5, bis sie akzeptiert werden

Hier sind die Schritte aufgeführt, die zum Aktualisieren des mit der Überprüfung verbundenen Regalsatzes erforderlich sind.

  1. Wählen Sie im Bereich "Code Review" den Link "View Shelveset"
  2. Markieren und kopieren Sie im Bereich "Shelveset-Details" den Namen des Shelvesets
  3. Navigieren Sie zum Bereich "Ausstehende Änderungen", klicken Sie auf "Ablage" und fügen Sie den Namen der Ablage ein
  4. Klicken Sie im Dialogfeld zum Ersetzen des Regalsatzes auf die Schaltfläche Ja
  5. Jetzt kann der Überprüfer die aktualisierten Dateien sehen und die Überprüfungsdiskussion kann fortgesetzt werden

Ich habe einige Screenshots beigefügt, da ich finde, dass dies zur Klärung der Dinge beiträgt.


1) Wählen Sie im Bereich "Code Review" den Link "View Shelveset" aus, wie hier gezeigt:

enter image description here


2) Markieren und kopieren Sie im Bereich "Shelveset Details" den Namen des Shelvesets wie hier gezeigt:

enter image description here


3) Navigieren Sie zum Bereich "Ausstehende Änderungen", klicken Sie auf "Ablage" und fügen Sie den Namen der Ablage ein, zum Beispiel:

enter image description here


4) Klicken Sie im Bestätigungsdialogfeld für das Ersetzen des Regalsatzes auf die Schaltfläche Ja:

enter image description here

150
Chad Dienhart

Ich glaube, dass die korrekte Vorgehensweise für Person 1 ist, um die Änderungen vorzunehmen und eine weitere Überprüfung anzufordern. Wenn Ihr Code Arbeit benötigt, bedeutet dies, dass Sie ihn ändern, sodass Sie die alte Version verwenden möchten, auf die Sie zum Vergleich zurückblicken können. Sie haben noch die alte Überprüfung in der Geschichte, nachdem es geschlossen wird, wenn Sie die Kommentare ansehen möchten. Wir sind derzeit dabei, unseren Codeüberprüfungsprozess an meinem Arbeitsplatz zu optimieren.

6
Paddy

Ich weiß, dass diese Frage alt ist, aber sie wird immer noch nicht von anderen Plakaten unterstützt. Die von chad vorgeschlagene Lösung wird für einige Dinge funktionieren, für andere jedoch ein merkwürdiges Verhalten aufweisen.

Vor kurzem begann das TFS-Team mit der Planung einer Lösung für eine sehr alte Benutzeranforderung, um die Aktualisierung einer Codeüberprüfung zu ermöglichen , die dieses Problem elegant lösen sollte, indem Codeüberprüfungen Iterationen enthalten.

2
julealgon

Sie müssen dies mit zwei verschiedenen Bewertungen tun. Es gibt aber auch eine Möglichkeit, die Geschichte mit der zweiten Überprüfung beizubehalten. Alles was Sie brauchen sind Aufgaben.

Dieser Workflow wird für Änderungssatz-basierte Überprüfungen beschrieben, funktioniert aber auch für Shelve-basierte Überprüfungen.

  1. Erstellen Sie task1
  2. Fügen Sie vor dem Einchecken von changeset1 task1 als zugehöriges Arbeitselement hinzu
  3. Einchecken von Änderungen mit einem bestimmten Arbeitselement und Anfordern einer Überprüfung dieses Änderungssatzes
  4. Erstellen Sie task2
  5. Fügen Sie vor dem Einchecken von changeset2 beide Aufgaben als zugehöriges Arbeitselement hinzu.
  6. Einchecken von Änderungen mit zwei zugehörigen Arbeitselementen und Überprüfen dieses Änderungssatzes anfordern

In der zweiten Überprüfungsanforderung kann der Überprüfer nun nach verwandten Aufgaben suchen. Wenn der Überprüfer nach Aufgabe1 sucht, werden das Änderungsset1 und die Überprüfungsanforderung mit ihren Kommentaren angezeigt. So verlieren Sie nicht den Gesprächsverlauf.

0
Ville Pelho