En nuestro proyecto, cada cambio significativo en el sistema es revisado por el líder del equipo o junto con otro desarrollador que será el principal "consumidor" del nuevo módulo. Hablamos sobre skype y usamos Rudel en Emacs (un complemento para edición colaborativa, básicamente permite que varios usuarios editen el mismo archivo en vivo), o TypeWith.me (Piratepad), o uno de nosotros comparte su pantalla en skype.
Es difícil cuantificar esto, porque no se revisan los cambios mundanos, como las nuevas vistas, páginas, etc. Revisamos nuevos módulos, actualizaciones importantes y refactorizaciones. En cuanto a los grandes cambios, la revisión del código puede llevar del 10% al 30% del tiempo, pero vale la pena.
Puedo decir que la programación en pareja, cuando 2 programadores editan el mismo archivo al mismo tiempo, no solo sentarse en la misma computadora, es mucho mejor que la práctica habitual de oficina de sentarse detrás del hombro.
Para cosas simples como convenciones de nomenclatura y errores de alcance, utilizamos nuestras propias herramientas automáticas de código abierto (jslint, pylint, pyflakes, pep8). Y no limitamos los commits y los push: utilizamos Mercurial, que tiene ramificaciones y fusiones muy fáciles (debo decir que es más fácil que en Git). Los errores no son una cuestión de revisión de código.
Hacemos reuniones de equipo donde se anuncian los cambios y las cosas nuevas, pero allí, no todos realmente prestan atención. Probablemente deberíamos hacer un poco más de revisiones de código.