Должны ли мы использовать «Отклонить» или «Ждать автора» в TFS, когда есть что исправить?


12

В TFS, когда мы помещаем комментарии для вещей, которые должны быть исправлены, в запрос на извлечение перед тем, как мы его примем, мы должны пометить его как Отклонить или Дождаться автора? Что лучше?

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

Ответы:


14

В соответствии с кодом Microsoft по проверке с запросами на извлечение: Голосуйте за изменения, предлагаемая цель каждого класса одобрения:

Утвердить с предложениями : согласитесь с запросом на получение ответа, но предоставьте дополнительные предложения по улучшению кода.

Ожидание автора : не одобряйте изменения и попросите автора проверить ваши комментарии. Автор должен сообщить вам, когда вам следует пересмотреть код после того, как они рассмотрят ваши проблемы.

Отклонено : изменения не принимаются. Если вы голосуете таким образом, вы должны оставить комментарий в запросе на получение ответа, в котором подробно объясняется, почему изменения были отклонены.

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

И это Rejectedозначает, что ни в коем случае вы не принимаете такие изменения, независимо от того, насколько хорошо написан код.

Вопрос, который вы должны задать себе, заключается в том, соответствует ли идея вашей группы предложенной цели идее Microsoft.


1
Об этом: «Автор должен сообщить вам, когда вы должны пересмотреть код после того, как они рассмотрят ваши проблемы». - существует ли механизм, позволяющий автору предупредить рецензентов о том, что ОР готов к повторной проверке?
Роберт Сим

@RobertSim - мы используем опцию «Пинг» в разделе «Рецензенты», чтобы отправить сообщение с просьбой о повторной проверке.
killercowuk

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