Carregando
A maioria dos code reviews e teatro. O dev olha por 5 minutos, marca approved e segue a vida. Aqui ta o checklist que realmente pega os bugs
Code Review que Pega Bug de Verdade: Checklist. A maioria dos code reviews e teatro. O dev olha por 5 minutos, marca approved e segue a vida. Aqui ta o checklist que realmente pega os bugs que importam — os que custam caro.
Vou ser direto: a maioria dos code reviews no mercado nao serve pra nada. O dev abre o PR, olha por 3 minutos, deixa um comentario sobre nome de variavel ou indentacao, marca approved e segue com a vida. Enquanto isso, um overflow silencioso ou uma race condition passa batido e vai pra producao.
Estudos da Microsoft Research mostram que reviews eficazes demoram entre 60 e 90 minutos por 200-400 linhas de codigo. Se voce ta revisando 2000 linhas em 15 minutos, voce nao ta revisando — ta fingindo. E ta tudo bem admitir isso. O primeiro passo pra melhorar e reconhecer que o processo atual nao funciona.
O problema real e que code review virou ritual. Algo que o time faz porque 'e boa pratica', nao porque acredita que vai encontrar bugs. E quando voce faz algo so por obrigacao, o resultado e proporcional ao esforco investido: zero.
Entao como transformar code review de teatro em algo que realmente protege o codigo? Dividindo em niveis. Nem todo PR precisa do mesmo nivel de scrutinio. Um fix de typo nao precisa da mesma atencao que uma mudanca no calculo de pagamento.