Devemos usar Rejeitar ou Aguardar autor no TFS quando houver coisas a serem corrigidas?

12

No TFS, quando colocamos comentários para que as coisas sejam corrigidas em uma solicitação de recebimento antes de aceitá-lo, devemos marcá-lo como Rejeitar ou Aguardar autor? Qual é melhor?

insira a descrição da imagem aqui

sashoalm
fonte

Respostas:

14

De acordo com o código Microsoft on Review com solicitações pull: votar nas alterações, o objetivo sugerido para cada classe de aprovação é:

Aprovar com sugestões : concorde com a solicitação de recebimento, mas forneça sugestões opcionais para melhorar o código.

Aguardando o autor : não aprove as alterações e peça ao autor para revisar seus comentários. O autor deve informar quando você deve revisar novamente o código depois que eles abordarem suas preocupações.

Rejeitado : as alterações não são aceitáveis. Se você estiver votando dessa maneira, deixe um comentário na solicitação de recebimento, detalhando por que as alterações foram rejeitadas.

Portanto, Waiting for Authorentendo que você acha que o autor estragou sua abordagem da solução, mas que seu código é resgatável se eles levam seus comentários a sério.

E Rejectedsignifica que, de jeito nenhum, você está aceitando mudanças como essa, não importa quão bem escrito seja o código.

A pergunta que você deve fazer é se a ideia do grupo sobre o objetivo sugerido corresponde à ideia da Microsoft.

Peter M
fonte
1
Sobre isso: "O autor deve informar quando você deve revisar novamente o código depois que eles abordarem suas preocupações". - existe um mecanismo para o autor alertar os revisores de que o PR está pronto para ser revisto?
Robert Sim
@RobertSim - usamos a opção 'Ping' em Revisores para enviar uma mensagem solicitando nova revisão.
killercowuk