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


12

In TFS, quando inseriamo i commenti per risolvere le cose in una richiesta pull prima di accettarla, dovremmo contrassegnarlo come Rifiuta o Attendi autore? Che è migliore?

inserisci qui la descrizione dell'immagine

Risposte:


14

Come da Microsoft sul codice di revisione con richieste pull: Vota sulle modifiche lo scopo suggerito di ogni classe di approvazione è:

Approvare con suggerimenti : concordare con la richiesta pull, ma fornire suggerimenti opzionali per migliorare il codice.

In attesa dell'autore : non approvare le modifiche e chiedere all'autore di rivedere i tuoi commenti. L'autore dovrebbe informarti quando è necessario rivedere il codice dopo aver risolto i tuoi dubbi.

Rifiutato : le modifiche non sono accettabili. Se stai votando in questo modo, dovresti lasciare un commento nella richiesta pull che spieghi in dettaglio perché le modifiche sono state respinte.

Quindi Waiting for Authorintendo che tu pensi che l'Autore abbia sbagliato nel suo approccio alla soluzione, ma che il suo codice sia riscattabile se prendono a cuore i tuoi commenti.

E Rejectedsignifica che all'inferno non accetti cambiamenti come questo, non importa quanto sia scritto bene il codice.

La domanda che ti devi porre è se l'idea dei tuoi gruppi dello scopo suggerito corrisponda all'idea di Microsoft.


1
A proposito di questo: "L'autore dovrebbe farti sapere quando dovresti rivedere il codice dopo aver affrontato le tue preoccupazioni". - esiste un meccanismo per l'autore per avvisare i revisori che il PR è pronto per la revisione?
Robert Sim,

@RobertSim: utilizziamo l'opzione "Ping" in Revisori per inviare un messaggio che chiede di riesaminarlo.
killercowuk,

Un altro modo di vedere questo: Waiting for Author= Voglio questa funzione, ma l'hai codificata in modo errato e non approverò fino a quando non la risolverai nel modo che suggerisco. E Rejected= Non voglio questa funzione che mi stai chiedendo di recensire.
goku_da_master il
Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.