Dans TFS, lorsque nous mettons des commentaires pour que les choses soient corrigées dans une demande de tirage avant de l'accepter, devons-nous le marquer comme rejet ou en attente d'auteur? Ce qui est mieux?
Selon Microsoft le Revoir le code avec les demandes de tirage: voter sur les modifications l'objectif suggéré de chaque classe d'approbation est:
Approuver avec des suggestions : d'accord avec la demande d'extraction, mais fournir 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.
Ainsi je prends Waiting for Author
pour signifier 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 Rejected
signifie que vous n'acceptez aucun changement comme celui-ci, peu importe à quel point 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.