Dovremmo usare Rifiuta o Attendi autore in TFS quando ci sono cose da risolvere?

2

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?

    
posta sashoalm 15.06.2017 - 13:06
fonte

1 risposta

5

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.

    
risposta data 15.06.2017 - 13:19
fonte

Leggi altre domande sui tag