La revisione del codice è una delle attività ad alto impatto per un TL: protegge la qualità e insegna. L'obiettivo è codice migliore e un team più forte, non gatekeeping. Le revisioni devono essere veloci, gentili e focalizzate su ciò che conta.
Principi
✓ 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
Un esempio concreto
Invece di "Questo è sbagliato, usa una map," scrivi: "nit: una map qui eviterebbe il ciclo annidato, ne vale la pena? Non sto bloccando." Stesso punto, ma insegna, lascia spazio per dissentire, e non blocca il merge.
