Faut-il utiliser Rejeter ou Attendre l'auteur dans TFS quand il y a des choses à corriger?

12

Dans TFS, lorsque nous mettons des commentaires pour que les choses soient corrigées dans une demande de traction avant de l'accepter, devons-nous la marquer comme rejet ou en attente d'auteur? Ce qui est mieux?

entrez la description de l'image ici

sashoalm
la source

Réponses:

14

Selon Microsoft sur le code de révision avec des demandes d'extraction: voter sur les modifications, l'objectif suggéré de chaque classe d'approbation est le suivant:

Approuver avec des suggestions : acceptez la demande d'extraction, mais fournissez des suggestions facultatives pour améliorer le code.

En attente de l'auteur : n'approuvez pas les modifications et demandez à l'auteur de revoir vos commentaires. L'auteur devrait vous faire savoir quand vous devriez revoir le code après avoir répondu à vos préoccupations.

Rejeté : les modifications ne sont pas acceptables. Si vous votez de cette façon, vous devez laisser un commentaire dans la demande d'extraction expliquant pourquoi les modifications ont été rejetées.

Je suppose Waiting for Authordonc que vous pensez que l'Auteur a bousillé son approche de la solution mais que son code est échangeable s'il prend à cœur vos commentaires.

Et cela Rejectedsignifie que vous n'acceptez aucun changement comme celui-ci, même si le code est bien écrit.

La question que vous devez vous poser est de savoir si l'idée que votre groupe a de l'objectif suggéré correspond à l'idée de Microsoft.

Peter M
la source
1
À propos de ceci: "L'auteur devrait vous faire savoir quand vous devriez revoir le code après avoir répondu à vos préoccupations." - Existe-t-il un mécanisme permettant à l'auteur d'avertir les examinateurs que le PR est prêt à être réexaminé?
Robert Sim
@RobertSim - nous utilisons est l'option «Ping» sous Reviewers pour envoyer un message demandant une révision.
killercowuk
Une autre façon de voir Waiting for Authorles choses : = Je veux cette fonctionnalité, mais vous l'avez mal codée, et je n'approuverai pas jusqu'à ce que vous la corrigiez comme je le suggère. Et Rejected= je ne veux pas de cette fonctionnalité que vous me demandez de revoir.
goku_da_master