A revisão de código é uma das atividades de maior alavancagem de um TL: protege a qualidade e ensina. O objetivo é código melhor e um time mais forte, não gatekeeping. As revisões devem ser rápidas, gentis e focadas no que importa.
Princípios
✓ Review the CODE, never the person ("this function" not "you")
✓ Distinguish must-fix from nice-to-have (label nits explicitly)
✓ Ask questions, don't issue commands ("what about X?" invites discussion)
✓ Praise good work, not just problems
✓ Be fast — a PR blocked for two days kills momentum
✓ Approve when it's good enough, not perfect
Um exemplo concreto
Em vez de "Isso está errado, use um map," escreva: "dica: um map aqui evitaria o loop aninhado, vale a pena? Não está bloqueando." Mesmo ponto, mas ensina, deixa espaço para discordância e não bloqueia o merge.
