Fai riunioni di revisione del codice di persona al tempo stabilito ogni settimana. Ho venduto questo al mio compagno di squadra in questo modo (siamo in realtà entrambi gli sviluppatori senior, ma qualunque cosa):
"La revisione del codice è parzialmente lì per farmi conoscere meglio il tuo codice e sapere cosa sta succedendo nel tuo lato delle cose nel caso in cui un giorno venga investito da un camion e mi viene ordinato di finire lo sprint. Ma principalmente è lì per te a spiegare il tuo codice a qualcun altro, perché quando lo fai, coinvolge una parte diversa del tuo cervello, e spesso la tua spiegazione per loro, e / o le loro domande o commenti, potrebbe farti ricordare qualcosa che hai dimenticato di fare nel codice, o potresti farti realizzare un modo migliore per renderlo più leggibile o meglio configurarlo, il che porta a un codice più bello. "
Mi piace pensarlo come un gioco di parole. Le persone possono mostrare il loro lavoro ai loro coetanei. Non si tratta dei tuoi pari che trovano cose sbagliate nel tuo lavoro, a cui a nessuno piace la sensazione. Si tratta di impressionare i tuoi coetanei con il tuo codice fantastico, che a tutti piace.
Comunque penso che usare strumenti di revisione del codice in cui non ci sia interazione umana, nessuna riunione in una stanza, nessuna lavagna ... diventa solo un'altra fastidiosa "cosa" da fare. Non è che non ci dovrebbero essere tali strumenti, ma dovrebbero essere qualcosa a cui ricorrere se, durante la riunione di revisione del codice, si è reso conto che potrebbe essere necessaria una revisione più approfondita di una determinata sezione di codice. Quindi potresti assegnare uno degli sviluppatori junior per rivedere il codice dell'altro su una determinata area.