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

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?

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:

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

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

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

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.

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.

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.