Czy w TFS powinniśmy użyć opcji Odrzuć lub Czekaj na autora, jeśli są rzeczy do naprawienia?

13

Czy w TFS, gdy umieszczamy komentarze do rzeczy, które mają zostać naprawione w żądaniu ściągnięcia, zanim je zaakceptujemy, czy powinniśmy oznaczyć je jako Odrzuć lub Czekaj na autora? Który jest lepszy?

wprowadź opis zdjęcia tutaj

sashoalm
źródło

Odpowiedzi:

15

Zgodnie z kodem recenzji Microsoft dotyczącym recenzji z żądaniami ściągania: głosuj na zmianach sugerowany cel każdej klasy zatwierdzenia to:

Zatwierdź za pomocą sugestii : Zgadzam się z żądaniem ściągnięcia, ale podaj opcjonalne sugestie, aby poprawić kod.

Oczekiwanie na autora : Nie zatwierdzaj zmian i poproś autora o przejrzenie twoich komentarzy. Autor powinien poinformować Cię, kiedy powinieneś ponownie zapoznać się z kodem po tym, jak wyraził swoje obawy.

Odrzucony : zmiany są niedopuszczalne. Jeśli głosujesz w ten sposób, powinieneś zostawić komentarz w żądaniu ściągnięcia, wyszczególniający, dlaczego zmiany zostały odrzucone.

Rozumiem zatem, Waiting for Authorże uważasz, że autor spieprzył swoje podejście do rozwiązania, ale jego kod można wykorzystać, jeśli wezmą twoje uwagi do serca.

I Rejectedoznacza, że nie ma mowy, ty zaakceptowaniem zmiany takiego nie ważne jak dobrze napisany kod jest.

Pytanie, które musisz sobie zadać, brzmi: czy Twój pomysł grupy na sugerowany cel pasuje do idei Microsoftu.

Peter M.
źródło
1
O tym: „Autor powinien poinformować Cię, kiedy powinieneś ponownie zapoznać się z kodem po tym, jak rozwiał Twoje wątpliwości”. - czy autor ma mechanizm powiadamiania recenzentów, że PR jest gotowy do ponownej oceny?
Robert Sim
@RobertSim - korzystamy z opcji „Ping” w sekcji Recenzenci, aby wysłać wiadomość z prośbą o ponowne sprawdzenie.
killercowuk
Inny sposób spojrzenia na to: Waiting for Author= Chcę tę funkcję, ale źle ją zakodowałeś i nie zatwierdzę, dopóki nie naprawisz jej tak, jak sugeruję. I Rejected= Nie chcę tej funkcji, którą prosisz mnie o sprawdzenie.
goku_da_master