No, non stai facendo correttamente le revisioni del codice (ovviamente!). Quindi la domanda rimane se c'è qualcosa di sbagliato nel tuo codice che ha bisogno di sessioni di revisione? (cioè se stai spedendo codice funzionante, le revisioni del codice sono una soluzione alla ricerca di un problema).
Se pensi che sarebbe utile, allora inizierei a lavorare con uno strumento di revisione del codice che puoi utilizzare in modalità "come e quando" - quindi carichi una richiesta di revisione del codice ma non si ferma il resto il processo di commit e costruzione di codice da accadere. Penso che se hai aggiunto la recensione al processo del flusso di lavoro che ha impedito i commit fino a dopo la revisione, finiresti per frustrarti e aggirare le recensioni. Sii onesto con te stesso in questo caso.
Ma se riesci a leggere il codice quando hai tempo, diventa un buon modo per mostrare cosa hai fatto per risolvere un compito di lavoro e puoi sempre correggere retroattivamente i difetti di revisione del codice sollevando un errore successivo.
Gli strumenti per la revisione del codice sono ReviewBoard - un sito Web di base Python. Carichi diffs al sito e puoi quindi assegnarli e commentarli. VMWare lo ha sviluppato per i propri scopi.
Gerrit funziona bene se usi git.
Redmine ha un buon plug-in di revisione del codice che consente recensioni basate su qualsiasi commit, basta creare una nuova recensione come solo un altro ticket tracker . È piacevole da usare e non ti ostacola.