Recenzje kodu TFS - pokaż zaktualizowane pliki w odpowiedzi na komentarze

127

Zaczynamy używać funkcji przeglądu kodu wbudowanej w wersję zapoznawczą VS 2012 i VS 2013. Poproszenie o sprawdzenie i dodanie komentarzy wydaje się całkiem proste. Jeśli ktoś dodaje komentarze z prośbą o zmianę kodu, to w jaki sposób żądający wprowadza te zmiany i je pokazuje?

Więc proces przebiegałby następująco:

  1. Osoba 1 prosi o sprawdzenie kodu.
  2. Osoba 2 dodaje komentarze i wybiera „Wymaga pracy”.
  3. Osoba 1 dokonuje niezbędnych zmian.

W jaki sposób Osoba 1 pokazuje teraz te zmiany Osobie 2? Możesz dodawać komentarze i wysyłać je, ale pliki się nie zmieniają. Zakładam, że pliki pochodzą z zestawu zmian utworzonego, gdy zażądano oryginalnej recenzji. Czy Osoba 1 powinna zamknąć tę recenzję i poprosić o drugą recenzję?

Utworzenie drugiej recenzji nie wydaje się optymalne, ponieważ tracisz historię rozmowy o tym, dlaczego wprowadzałeś zmiany.

Istnieje mnóstwo witryn internetowych pokazujących, jak korzystać z podstawowych funkcji recenzji kodu, ale czy są jakieś witryny, które przedstawiają najlepsze praktyki? Na przykład, kto powinien klikać pola wyboru obok plików?

Jan
źródło
+1 dla „czy są jakieś witryny, które stosują sprawdzone metody”. ALM Rangers są zwykle moimi „goto” do dogłębnego wykorzystania, ale jedyna informacja, jaką mogę znaleźć, nie odpowiada na twoje pytanie. Chciałbym zobaczyć kilka poradników i scenariuszy korzystania z Code Review.
DaveShaw,

Odpowiedzi:

164

Więc proces przebiegałby następująco:

  1. Osoba 1 prosi o sprawdzenie kodu.
  2. Osoba 2 dodaje komentarze i wybiera „Wymaga pracy”.
  3. Osoba 1 dokonuje niezbędnych zmian.
  4. Osoba 1 Aktualizuje zestaw półek powiązany z przeglądem kodu
  5. Osoba 1 dodaje komentarze, aby kontynuować dyskusję
  6. Powtarzaj kroki 2–5, aż zostaną zaakceptowane

Oto kroki niezbędne do zaktualizowania zestawu półek powiązanego z recenzją.

  1. W okienku „Przegląd kodu” wybierz link „Wyświetl zestaw półek”
  2. W okienku „Szczegóły zestawu półek” zaznacz i skopiuj nazwę zestawu półek
  3. Przejdź do okienka „Oczekujące zmiany”, kliknij „Półka” i wklej nazwę zestawu półek
  4. Naciśnij przycisk Tak w oknie dialogowym weryfikacji wymiany zestawu półek
  5. Teraz recenzent może zobaczyć zaktualizowane pliki, a dyskusja na temat recenzji może być kontynuowana

Dołączyłem kilka zrzutów ekranu, ponieważ uważam, że pomaga to wyjaśnić.


1) W okienku „Przegląd kodu” wybierz łącze „Wyświetl zestaw półek”, jak pokazano poniżej:

wprowadź opis obrazu tutaj


2) W okienku „Szczegóły zestawu półek” zaznacz i skopiuj nazwę zestawu półek, jak pokazano tutaj:

wprowadź opis obrazu tutaj


3) Przejdź do okienka „Oczekujące zmiany”, kliknij „Półka” i wklej nazwę zestawu półek, na przykład:

wprowadź opis obrazu tutaj


4) Naciśnij przycisk Tak w oknie dialogowym weryfikacji wymiany zestawu półek:

wprowadź opis obrazu tutaj

Chad Dienhart
źródło
7
Problem z tym rozwiązaniem polega na tym, że jest ono destrukcyjne, ponieważ wszystkie komentarze są zdmuchnięte. Myślę, że lepszym rozwiązaniem byłoby utworzenie nowej recenzji kodu i ustawienie starego jako powiązanego elementu.
Bardia
15
Wymiana zestawu półek nie dotyczy komentarzy tylko plików. Nasz zespół z powodzeniem stosuje tę metodę i nie tracimy żadnych komentarzy do recenzji.
Chad Dienhart
10
To działa, ale jeśli dodasz / usuniesz linie podczas edycji pliku i zastąpienia zestawu półek, komentarze, które znajdowały się w tym pliku, będą wskazywać na niewłaściwe wiersze.
Chin
2
Zwykle nie lubię powtarzać, ale powyższy punkt @ Chin sprawia, że ​​ta odpowiedź jest całkowicie nie do utrzymania. Tak, możesz łatwo zaktualizować swój zestaw półek, ale w ten sposób zasadniczo WSZYSTKIE poprzednie komentarze do recenzji wskazują teraz NIEPRAWIDŁOWY tekst. (Ponieważ najwyraźniej komentarz do recenzji jest dołączony do absolutnego przesunięcia od początku pliku.)
Michael Sorens
7
Zbyt ręczny i nie idealny. TFS lub Visual Studio powinny mieć coś do porównania zmian między kolejnymi recenzjami, aby recenzent mógł zobaczyć dokładnie zmiany wprowadzone od ostatniej recenzji. Nie istnieje coś takiego? Może wtyczka czy coś ...
Iúri dos Anjos,
6

Uważam, że właściwą procedurą jest dla Osoby 1 wprowadzenie zmian i zażądanie kolejnej oceny. Kiedy twój kod wymaga pracy, oznacza to, że będziesz go zmieniać, więc będziesz chciał mieć starą wersję, aby spojrzeć wstecz w celu porównania. Jeśli chcesz przejrzeć komentarze, nadal masz w historii starą recenzję po jej zamknięciu. Obecnie jesteśmy w trakcie optymalizacji naszego procesu przeglądu kodu w moim miejscu pracy.

Paddy
źródło
2

Wiem, że to pytanie jest stare, ale nadal nie jest obsługiwane, jak wskazują inne plakaty. Rozwiązanie zaproponowane przez czada zadziała w niektórych przypadkach, ale będzie miało dziwne zachowanie w przypadku innych.

Niedawno zespół TFS rozpoczął etapy planowania w celu rozwiązania bardzo starego żądania opinii użytkownika, aby umożliwić aktualizację przeglądu kodu, który powinien elegancko rozwiązać ten problem, wprowadzając iteracje do przeglądów kodu.

julealgon
źródło
1
Wspaniały. Ja też nie mogę się doczekać. Byłbym zadowolony z prawdziwego rozwiązania. Mój post to po prostu sposób na obejście problemu.
Chad Dienhart
Miejmy nadzieję, że nie będzie wyglądać jak lazurowe żądania ściągnięcia DevOps. Są koszmarem spamerskim.
StingyJack
0

Musisz to zrobić z dwoma różnymi recenzjami. Ale jest też sposób na zachowanie historii przy drugiej recenzji. Potrzebujesz tylko zadań.

Ten przepływ pracy jest opisany dla recenzji opartych na zestawie zmian, ale działa również w przypadku recenzji opartych na półce.

  1. Utwórz zadanie 1
  2. Przed wpisaniem zestawu zmian1 dodaj zadanie1 jako powiązany element pracy
  3. Wpisz zmiany związane z jednym elementem pracy i poproś o przegląd tego zestawu zmian
  4. Utwórz zadanie 2
  5. Przed wpisaniem zmian w zestawie2 dodaj oba zadania jako powiązane elementy pracy.
  6. Wpisz zmiany z dwoma powiązanymi elementami pracy i poproś o przegląd tego zestawu zmian

Teraz w drugiej prośbie o recenzję recenzent może szukać powiązanych zadań, a jeśli przeglądający szuka zadania 1, widzi zestaw zmian1 i żądanie przeglądu wraz z komentarzami. Więc nie stracisz historii rozmowy.

Ville Pelho
źródło