В TFS, когда мы помещаем комментарии для вещей, которые должны быть исправлены, в запрос на извлечение перед тем, как мы его примем, мы должны пометить его как Отклонить или Дождаться автора? Что лучше?
В соответствии с кодом Microsoft по проверке с запросами на извлечение: Голосуйте за изменения, предлагаемая цель каждого класса одобрения:
Утвердить с предложениями : согласитесь с запросом на получение ответа, но предоставьте дополнительные предложения по улучшению кода.
Ожидание автора : не одобряйте изменения и попросите автора проверить ваши комментарии. Автор должен сообщить вам, когда вам следует пересмотреть код после того, как они рассмотрят ваши проблемы.
Отклонено : изменения не принимаются. Если вы голосуете таким образом, вы должны оставить комментарий в запросе на получение ответа, в котором подробно объясняется, почему изменения были отклонены.
Таким образом, я понимаю, Waiting for Authorчто вы думаете, что Автор испортил свой подход к решению, но что его / ее код подлежит обмену, если они примут ваши комментарии близко к сердцу.
И это Rejectedозначает, что ни в коем случае вы не принимаете такие изменения, независимо от того, насколько хорошо написан код.
Вопрос, который вы должны задать себе, заключается в том, соответствует ли идея вашей группы предложенной цели идее Microsoft.
Waiting for Author= Я хочу эту функцию, но вы ее неправильно закодировали, и я не одобряю, пока вы не исправите ее так, как я предлагаю. ИRejected= я не хочу эту функцию, которую вы просите меня рассмотреть.