TFS-Codeüberprüfungen - Zeigen Sie aktualisierte Dateien als Antwort auf Kommentare an

127

Wir beginnen, die in VS 2012 und VS 2013 integrierte Codeüberprüfungsfunktion zu verwenden. Das Anfordern der Überprüfung und das Hinzufügen von Kommentaren scheint 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 folgendermaßen ablaufen:

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

Wie zeigt Person 1 diese Änderungen nun an 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 Best Practices aufgeführt sind? Wer sollte beispielsweise die Kontrollkästchen neben Dateien aktivieren?

John
quelle
+1 für "Gibt es Websites, auf denen die Best Practices angezeigt werden?". Die ALM Rangers sind normalerweise meine "Goto-Typen" für die eingehende Verwendung, aber die einzigen Informationen, die ich finden kann, beantworten Ihre Frage nicht. Ich würde gerne einige Walkthoughs und Szenarien für die Verwendung von Code Review sehen.
DaveShaw

Antworten:

164

Der Prozess würde also folgendermaßen ablaufen:

  1. Person 1 fordert eine Codeüberprüfung an.
  2. Person 2 fügt Kommentare hinzu und wählt "Needs Work" aus.
  3. Person 1 nimmt die notwendigen Änderungen vor.
  4. Person 1 Aktualisiert das mit der Codeüberprüfung verknüpfte Regalset
  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überprüfung" den Link "Regalsatz anzeigen"
  2. Markieren und kopieren Sie im Bereich "Regaldetails" den Namen des Regalsatzes
  3. Navigieren Sie zum Bereich "Ausstehende Änderungen", klicken Sie auf "Regal" und fügen Sie den Namen des Regalsatzes ein
  4. Klicken Sie im Dialogfeld zur Überprüfung des Regalsatzes auf die Schaltfläche Ja
  5. Jetzt kann der Prüfer die aktualisierten Dateien sehen und die Besprechungsdiskussion kann fortgesetzt werden

Ich habe einige Screenshots beigefügt, da ich finde, dass es hilfreich ist, die Dinge zu klären.


1) Wählen Sie im Bereich "Codeüberprüfung" den Link "Regalsatz anzeigen" wie hier gezeigt:

Geben Sie hier die Bildbeschreibung ein


2) Markieren Sie im Bereich "Regaldetails" den Namen des Regalsatzes und kopieren Sie ihn wie hier gezeigt:

Geben Sie hier die Bildbeschreibung ein


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

Geben Sie hier die Bildbeschreibung ein


4) Klicken Sie im Dialogfeld zur Überprüfung des Regalsatzes auf die Schaltfläche Ja:

Geben Sie hier die Bildbeschreibung ein

Chad Dienhart
quelle
7
Das Problem bei dieser Lösung ist, dass es destruktiv ist, dass alle Kommentare weggeblasen werden. Ich denke, eine bessere Lösung wäre, eine neue Codeüberprüfung zu erstellen und die alte als verwandtes Element festzulegen.
Bardia
15
Das Ersetzen des Regalsatzes berührt nicht die Kommentare, sondern nur die Dateien. Unser Team verwendet diese Methode erfolgreich und wir verlieren keine Bewertungskommentare.
Chad Dienhart
10
Dies funktioniert, aber wenn Sie beim Bearbeiten einer Datei und beim Ersetzen des Regalsatzes Zeilen hinzufügen / entfernen, verweisen die Kommentare in dieser Datei auf die falschen Zeilen.
Chin
2
Normalerweise wiederhole ich das nicht gern, aber der obige Punkt von @ Chin macht diese Antwort völlig unhaltbar. Ja, Sie können Ihr Regalset problemlos aktualisieren, aber dadurch werden im Wesentlichen ALLE vorherigen Überprüfungskommentare angezeigt, die jetzt auf FALSCHEN Text verweisen. (Weil anscheinend ein Überprüfungskommentar an einen absoluten Versatz vom Anfang einer Datei angehängt ist.)
Michael Sorens
7
Zu manuell und nicht ideal. TFS oder Visual Studio sollten etwas zum Vergleichen der Änderungen zwischen aufeinanderfolgenden Überprüfungen haben, damit der Überprüfer genau die Änderungen sehen kann, die seit der letzten Überprüfung vorgenommen wurden. Gibt es so etwas nicht? Vielleicht ein Plugin oder so ...
Iúri dos Anjos
6

Ich glaube, das richtige Verfahren ist für Person 1, 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 zum Vergleich benötigen. Sie haben immer noch die alte Überprüfung in der Historie, nachdem sie geschlossen wurde, wenn Sie die Kommentare überprüfen möchten. Wir sind derzeit dabei, unseren Codeüberprüfungsprozess an meinem Arbeitsplatz zu optimieren.

Paddy
quelle
2

Ich weiß, dass diese Frage alt ist, aber sie wird immer noch nicht unterstützt, wie von anderen Postern hervorgehoben. Die von chad vorgeschlagene Lösung funktioniert für einige Dinge, für andere jedoch merkwürdig.

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

julealgon
quelle
1
Toll. Ich werde mich auch darauf freuen. Ich würde eine echte Lösung begrüßen. Mein Beitrag ist einfach eine Möglichkeit, das Problem zu umgehen.
Chad Dienhart
Hoffentlich sieht es nicht nach Azure DevOps-Pull-Anforderungen aus. Sie sind ein Spam-Albtraum.
StingyJack
0

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 auf Änderungssätzen basierende Überprüfungen beschrieben, funktioniert jedoch auch für auf Regalen basierende Überprüfungen.

  1. Aufgabe1 erstellen
  2. Fügen Sie vor dem Einchecken von Änderungssatz1 Aufgabe1 als zugehöriges Arbeitselement hinzu
  3. Checken Sie Änderungen mit einem Arbeitselement ein und fordern Sie eine Überprüfung dieses Änderungssatzes an
  4. Task2 erstellen
  5. Fügen Sie vor dem Einchecken von Changeset2 beide Aufgaben als zugehöriges Arbeitselement hinzu.
  6. Checken Sie Änderungen mit zwei zugehörigen Arbeitselementen ein und fordern Sie eine Überprüfung dieses Änderungssatzes an

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

Ville Pelho
quelle