Sembra che alla squadra manchi un processo formale per le revisioni del codice.
Non sto parlando di creare un documento Word di 350 pagine, ma solo alcuni semplici punti elenco su ciò che il processo comporta.
I bit importanti:
-
Definire un nucleo di revisori. Nessuna dichiarazione generale Assegna un nome alle persone.
Questi dovrebbero essere i tuoi sviluppatori senior.
-
Richiede più di 1 revisore del nucleo a firmare la recensione.
-
Identifica almeno un altro revisore non principale per ogni sprint o release che è un revisore nucleo temporaneo. Richiedere l'approvazione per tutte le revisioni del codice durante questo periodo.
L'elemento n. 3 consente agli altri sviluppatori di ruotare all'interno del gruppo di revisori nucleo. Alcune settimane passeranno più tempo sulle recensioni di altri. È un atto di bilanciamento.
Come ricompensare le persone? Molte volte riconoscere lo sforzo che una persona sta compiendo durante la revisione del codice di fronte a tutto il team può funzionare, ma non stressarti su questo.
In caso di dubbio, definisci il processo e comunica al team di cui hanno bisogno per attenervisi.
E c'è un'ultima cosa che puoi provare - per quanto possa essere controverso: lascia che il @ # $% colpisca il fan, se posso usare un idioma.
Lascia che il team fallisca, perché il processo di revisione del codice non viene seguito. La gestione sarà coinvolta, e quindi le persone cambieranno. Questa è davvero solo una buona idea nei casi più estremi in cui hai già definito un processo e il team si è rifiutato di rispettarlo. Se non hai l'autorità per licenziare o disciplinare le persone (come la maggior parte degli sviluppatori leader non ), devi coinvolgere qualcuno che possa fare queste cose.
E non c'è niente come l'incapacità di far cambiare le cose. Nonostante ciò che la gente potrebbe dire, puoi guidare il Titanic - ma non prima che colpisca il ghiaccio.
A volte devi solo lasciare che il Titanic colpisca il ghiaccio.