Чи варто використовувати відхилити чи чекати автора в TFS, коли потрібно виправити речі?


13

У TFS, коли ми ставимо коментарі до речей, які мають бути зафіксовані у запиті на виклик, перш ніж ми приймемо його, чи слід відзначати його як відхилити чи чекати автора? Який краще?

введіть тут опис зображення

Відповіді:


15

Відповідно до коду Microsoft on Review із запитами на виклик: Голосуйте за зміну запропонованої мети кожного класу схвалення:

Підтвердити пропозиції : Погодьтеся із запитом на витяг, але надайте необов'язкові пропозиції щодо вдосконалення коду.

Чекаємо автора : Не схвалюйте зміни та попросіть автора переглянути ваші коментарі. Автор повинен повідомити вам, коли вам слід переглянути код після того, як вони вирішили ваші проблеми.

Відхилено : зміни не прийнятні. Якщо ви голосуєте таким чином, ви повинні залишити коментар у запиті на виклик із детальним описом, чому зміни було відхилено.

Таким чином, я Waiting for Authorмаю на увазі, що ви думаєте, що автор заблукав у своєму підході до рішення, але що його / її код можна викупити, якщо вони приймуть до уваги ваші коментарі.

Це Rejectedозначає, що в пеклі ви не приймаєте подібних змін, незалежно від того, наскільки добре написаний код.

Питання, яке ви повинні задати собі, - чи відповідає ідея вашої групи щодо запропонованої мети ідеї Microsoft.


1
Про це: "Автор повинен повідомити вам, коли ви повинні переглядати код після того, як вони вирішили ваші проблеми". - чи існує механізм, щоб автор попереджав рецензентів про те, що PR готовий до перегляду?
Роберт Сім

@RobertSim - ми використовуємо опцію 'Ping' у розділі Рецензенти, щоб надіслати повідомлення з проханням про повторний розгляд.
killercowuk

Ще один спосіб подивитися на це: Waiting for Author= Я хочу цю функцію, але ви її неправильно зашифрували, і я не схвалюю, поки ви не виправте її так, як я пропоную. І Rejected= я не хочу, щоб ця функція ви просили переглянути.
goku_da_master
Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.