La revisión de código es una de las actividades con mayor impacto para un Tech Lead: protege la calidad y enseña. El objetivo es código mejor y un equipo más fuerte, no gatekeeping. Las revisiones deben ser rápidas, amables y enfocadas en lo que importa.
Principios
text
✓ 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 ejemplo concreto
En lugar de "Esto es incorrecto, usa un map", escribe: "nit: un map aquí evitaría el bucle anidado, ¿vale la pena? No lo bloqueo." Mismo punto, pero enseña, deja espacio para estar en desacuerdo y no bloquea el merge.