Stiamo provando la revisione obbligatoria del codice su ogni commit: non c'è nulla che arrivi al master che non sia stato convalidato da almeno 1 persona e non dall'autore - per un paio di sprint. Abbiamo acquistato da entrambi gli sviluppatori e il management (che è una situazione eccezionale) e vogliamo ottenere alcuni dei vantaggi per cui è noto:
- ovvia riduzione dei bug
- una maggiore consapevolezza dei cambiamenti che si verificano attorno al progetto
- il "so che qualcuno lo guarderà quindi non sarò pigro" / effetto anti-cowboy
- maggiore coerenza all'interno / tra progetti
Ma stiamo introducendo qualcosa che è conosciuto per ridurre la velocità, e se fatto male potrebbe creare uno stupido passaggio burocratico nella pipeline del commit che non fa altro che occupare del tempo. Cose di cui sono preoccupato:
- recensioni relative alla selezione nitida
- (iperbolicamente) persone che aprono enormi problemi architettonici come parte di una revisione del commit in due righe.
- Non voglio influenzare le risposte con altre cose.
Mentre siamo tutti persone ragionevoli e faremo molte analisi di sé, potremmo sicuramente utilizzare alcune informazioni acquisite in battaglia su quali tipi di cose dovremmo cercare di ottenere in una sessione di revisione per fare davvero recensioni lavora per noi. Quali sono alcune linee guida e criteri che hai trovato funzionanti?