Code review vira teatro quando um PR de 900 linhas passa em dois minutos. Você recebe aprovação, nenhum defeito cai e ninguém aprende nada sobre o sistema.
As regras de tamanho e prazo
PR de até 400 linhas: Michael Fagan mediu inspeções formais na IBM e publicou o método no IBM Systems Journal (1976). O revisor encontra mais defeito quando examina bloco pequeno, então você divide a feature antes de abrir o PR. Resposta em 4 horas úteis: o autor espera pouco e o contexto continua fresco. Dois revisores quando o código é sensível: billing, autenticação e migração de dado. O resto recebe um revisor.
O que o revisor comenta
Você aponta nome que esconde intenção, caminho de erro ignorado e teste ausente para o caso novo. Robert C. Martin trata esses pontos em Clean Code (2008). O revisor sugere o caminho e explica o risco; ele não reescreve o código do autor. O time troca a ordem das revisões, porque Google descreve em Software Engineering at Google (O'Reilly, 2020) que o revisor aprende o sistema junto com o autor.
O erro de gestão
Você não mede produtividade pelo tempo de revisão. Quem quer bater meta aprova rápido e o cuidado some. Meça o tempo de ciclo do PR e os defeitos em produção: os dois números dizem se a revisão funciona.
