In TFS, quando inseriamo i commenti affinché le cose vengano risolte in una richiesta pull prima di accettarla, dovremmo contrassegnarla come Rifiuta o Attendi autore? Quale è meglio?
Come da Microsoft su Codice di revisione con richieste di pull: Vota sulle modifiche lo scopo suggerito di ciascuna classe di approvazione è:
Approve with suggestions : Agree with the pull request, but provide optional suggestions to improve the code.
Waiting for the author : Do not approve the changes, and ask the author to review your comments. The author should let you know when you should re-review the code after they have addressed your concerns.
Rejected : The changes aren't acceptable. If you are voting this way, you should leave a comment in the pull request detailing why the changes were rejected.
Quindi prendo Waiting for Author
per indicare che pensi che l'Autore abbia sbagliato nel suo approccio alla soluzione, ma che il suo codice sia riscattabile se tieni a cuore i tuoi commenti.
E Rejected
significa che in nessun modo all'inferno accetti qualsiasi cambiamento come questo indipendentemente dal codice scritto.
La domanda che devi porsi è se l'idea dei gruppi dello scopo suggerito corrisponda all'idea di Microsoft.
Leggi altre domande sui tag pull-requests team-foundation-server