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?
fonte
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?
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 Author
entendo 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 Rejected
significa 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.