devterview_$ iniciar simulação

O que priorizar num code review

MédioPlenoProcesso

Pergunta

Você tem 20 minutos pra revisar um PR. Em que ordem de prioridade você olha as coisas?

Resposta esperada

Do que mais dói pra o que menos: (1) Está correto? Faz o que se propõe, trata os casos de borda, não introduz um bug óbvio. (2) Tem teste que prova isso e que vai pegar uma regressão? (3) Segurança e dados: input não validado, permissão, algo sensível em log, query sem parametrizar. (4) O design cabe no resto? Fronteiras, acoplamento, se não está reimplementando algo que já existe. (5) Legibilidade: nomes, funções, o próximo dev entende. (6) Nits de estilo — e esses, idealmente, o linter/formatter pega sozinho, não a pessoa. Além da ordem: comente com o 'porquê' e como classificação ('bloqueia' vs 'sugestão' vs 'curiosidade'), aprove com sugestões pequenas em vez de segurar o PR, e revise cedo (PR pequeno) — review de 800 linhas vira 'looks good'.

Por que perguntam isso

Pergunta pleno. Sinais de experiência: 'nits são do linter', classificar comentários, 'PR de 800 linhas vira looks good'.

#code-review#processo#prioridade
publicidade

Relacionadas