Code Review
Para quien revisa
⏱️Revisar en menos de 16 horas hábiles para no bloquear al compañero.
💬Los comentarios deben ser constructivos y específicos. Sugerir mejoras si algo está mal.
🔗Verificar que el MR referencia correctamente el workitem en el título.
✅Aprobar solo si el código es correcto, legible, no rompe nada existente y el CI está en verde.
🚫MRs extensos (>400 líneas): evaluar si puede dividirse. Si no, dedicar tiempo suficiente para una revisión rigurosa.
Para el autor del MR
🔬Auto-revisión: ser el primero en detectar errores obvios antes de asignar el MR.
📖Completar el template de descripción. Documentar en el workitem los pasos para probar.
🔄Mantener la rama actualizada con main (usando merge, no rebase) antes de incorporar correcciones.
🤝Responder todos los comentarios antes de pedir re-review.
Escalamiento de bloqueos
Si un MR se encuentra bloqueado por falta de review:
1️⃣Notificar al Team Leader.
2️⃣Si no se resuelve en un plazo prudente, escalar al Project Manager.
