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?
źródło
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?
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 Rejected
oznacza, ż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.
Waiting for Author
= Chcę tę funkcję, ale źle ją zakodowałeś i nie zatwierdzę, dopóki nie naprawisz jej tak, jak sugeruję. IRejected
= Nie chcę tej funkcji, którą prosisz mnie o sprawdzenie.