Guarda come Facebook fa con la propria app, chiamata phabricator: link
In pratica si basano su una base per numero e per ogni problema viene mostrato il codice, che deve essere esaminato da qualcuno. Il codice non entra nel loro repository principale fino a quando il revisore non ha detto che è giusto farlo.
Immagino che lo renda più divertente.
Inoltre, forse un codice dovrebbe essere assegnato a due persone: uno che lo fa e uno che lo controlla.
Anche se forse i tuoi compagni di squadra non credono in questa recensione.
Personalmente, in mancanza di revisori, ho usato test unitari per funzioni di livello inferiore e "test del custode" per tutto il resto: il test bidello è chiamato così, perché anche il bidello dovrebbe essere in grado di capire il tuo codice.
Di solito rimuovevo alcune parti minori, come le parentesi di blocco / funzione, le notazioni di visibilità, a volte persino i tipi, e lo mostravo a manager, esperti di dominio, compagni, chiunque richiedesse il codice: "è questo che vuoi?"
Inoltre, andare lì personalmente e non partire fino a quando la revisione non è stata completata aiuta
Oppure, nel caso in cui non stia bene con il team, o non stia bene con te, sai, "se puoi 'cambiare la compagnia, cambia compagnia" ...