The code is dark and full of errors

![[06-attachments/talks/2018-02-21-the-code-is-dark-and-full-of-errors/2018-02-21-the-code-is-dark-and-full-of-errors.png]]

Esta es la historia de Targaryen Consulting, una consultora de los Siete Reinos, ágil y con devoción por las buenas prácticas. Pero ni los dragones ni el acero valyrio la salvaron de una guerra civil: una guerra por culpa de los Code Reviews.

El origen del conflicto

Todo comenzó con una implementación forzada de un proceso de revisión sin objetivos claros:

• Revisiones eternas que bloqueaban deploys
• Comentarios pasivo-agresivos
• Criterios subjetivos que cambiaban según quién revisara
• Egos más grandes que los muros de Invernalia

Daenerys y Jon Snow, dos desarrolladores brillantes, terminaron enfrentados. El código se volvió oscuro. Los errores, abundantes. La moral, por el suelo.

¿Qué es (y qué no es) un Code Review?

• No es una auditoría personal
• Sí es una herramienta para mejorar la calidad colectiva del producto
• No es un checklist burocrático
• Sí es un espacio de aprendizaje, feedback y colaboración

Partes de un buen Code Review

• Objetivo claro: ¿qué se espera revisar?
• Alcance limitado: PRs pequeños y bien enfocados
• Contexto explícito: descripción, ticket, screenshots o capturas si aplica
• Comentarios con intención: sugerencias, preguntas, no juicios
• Reglas acordadas por el equipo

Buenas prácticas

• Revisá con empatía, no con soberbia
• Usá el chat para aclarar malentendidos, no para discutir en público
• Automatizá lo obvio (linting, tests), y enfocá la revisión en lo importante
• Agradecé los aportes y sugerencias, aunque no las apliques
• Iterá el proceso con retroalimentación del equipo

Key Takeaways

1. Un Code Review mal planteado puede destruir equipos.
2. La calidad del código es responsabilidad colectiva, no individual.
3. La empatía y el contexto valen más que la línea perfecta de código.
4. La cultura de revisión define la cultura técnica de un equipo.
5. Evitá convertir tu equipo en una lucha de clanes. No sos el Lord del código.

Conclusión

Quizás no sepamos si Daenerys y Jon Snow hicieron las paces.
Pero sí sabemos que una buena práctica de Code Reviews puede salvar más reinos de lo que pensás.
Porque el código puede ser oscuro y estar lleno de errores, pero con un buen proceso, todos podemos salir ilesos.