Revisione del codice e linee guida per il check-in: i revisori hanno il potere di veto sul check-in? [chiuso]

Naviga le tue risposte
2

Dovrebbe essere richiesto all'autore di un changeset di ottenere l'approvazione da tutti i revisori che hanno esaminato il codice o il codice dovrebbe essere autorizzato a effettuare il check-in finché è presente un unico accesso? Penso che dare il potere di veto ai revisori faccia più male che bene di come lo usano per ricattare (chiedendo modifiche inutili) e bloccare i check-in, ma saremmo curiosi di sentire l'opinione della gente su questa questione su quale sia la migliore pratica da seguire. / p>

EDIT: Essendo più specifico e al punto, dovrebbe essere bloccato un check se:

  1. ci sono alcune persone (una o più) che hanno firmato
  2. ma ci sono altri che hanno contrassegnato il diff come necessario per la revisione
  3. e l'autore ha fatto uno sforzo in buona fede affrontando i commenti dei revisori che hanno contrassegnato il diff come necessario alla revisione ma non possono farlo vieni d'accordo con loro?

cosa pensa la gente?

    
posta morpheus 30.03.2016 - 01:32
fonte

2 risposte

7

Non ha molta importanza. Se un collega "ricattasse" fuori dal raggio di azione appropriato del suo lavoro di revisione, lo porterebbe con il tuo manager o capo del team e lo risolverà in questo modo. Nessun processo impedirà a qualcuno con quel tipo di attitudine alla vita di provare a farti prendere in giro.

    
risposta data 30.03.2016 - 01:57
fonte
6

Dipende dal tuo ambiente. In alcuni ambienti, ho visto i revisori avere il potere di veto, e questo era importante dal momento che il livello di abilità degli sviluppatori (e l'impatto se un bug ce l'aveva fatta) lo rendeva necessario. Per la maggior parte degli ambienti, non credo che le revisioni del codice siano necessarie per l'80% + dei check-in. Per alcuni ambienti, ho visto i lead del team avere il potere di veto poiché in quegli ambienti, erano ritenuti insolitamente responsabili per problemi nel loro team.

Ma soprattutto, siamo tutti professionisti. Se qualcuno vede nella revisione del codice qualcosa che ritengono sia un problema, allora il team dovrebbe essere in grado di raggiungere un certo consenso su ciò che dovrebbe essere fatto. Se la squadra non riesce a raggiungere un certo consenso, non prenderei in considerazione il problema così grave da bloccare il check-in. Interrompi vendita di biciclette e concentrati su ciò che conta.

    
risposta data 30.03.2016 - 01:42
fonte

Leggi altre domande sui tag

Come sincronizzi le modifiche ai dati nel tuo database? [chiuso] Perché la maggior parte della definizione della sottostringa della lingua consente la sottostringa ("abc", 3)=""