5 min lectura • Guide 630 of 877
Mejores Prácticas de Code Review
Los code reviews efectivos equilibran rigurosidad con velocidad, detectando problemas importantes mientras evitan cuellos de botella de revisión que ralentizan la entrega. La integración de GitScrum con herramientas de code review ayuda a los equipos a rastrear el estado de revisión, identificar cuellos de botella y asegurar que las revisiones ocurran prontamente sin bloquear desarrolladores.
Diseño del Proceso de Review
SLAs de Review
NIVELES DE SERVICIO DE CODE REVIEW:
┌─────────────────────────┬─────────────┬─────────────────────┐
│ MÉTRICA │ OBJETIVO │ ACCIÓN SI NO SE CUMPLE│
├─────────────────────────┼─────────────┼─────────────────────┤
│ Primera respuesta │ < 4 horas │ Recordatorio en Slack│
│ Review completo │ < 24 horas │ Escalar a lead │
│ Merge después aprobación│ < 2 horas │ Notificar a autor │
│ Respuesta a revisión │ < 4 horas │ Bump de prioridad │
└─────────────────────────┴─────────────┴─────────────────────┘
Rastrear en GitScrum:
• Dashboard muestra tiempo promedio de review
• Alertas para PRs estancados
• Métricas de equipo para velocidad de review
Guías de Tamaño de PR
TAMAÑO ÓPTIMO DE PR:
┌───────────────┬─────────────┬───────────────────────────────┐
│ LÍNEAS CÓDIGO │ TIEMPO REV. │ DETECCIÓN DE DEFECTOS │
├───────────────┼─────────────┼───────────────────────────────┤
│ < 200 │ 15-30 min │ Alta - revisión rigurosa │
│ 200-400 │ 30-60 min │ Buena - revisión efectiva │
│ 400-800 │ 1-2 horas │ Media - fatiga del revisor │
│ > 800 │ 2+ horas │ Baja - revisión superficial │
└───────────────┴─────────────┴───────────────────────────────┘
ESTRATEGIAS PARA PRs MÁS PEQUEÑOS:
• Enviar refactoring separado de features
• Dividir features grandes en cambios incrementales
• Usar feature flags para implementaciones parciales
• Separar adiciones de tests de cambios de código
Calidad del Review
Qué Buscar
CHECKLIST DE CODE REVIEW:
┌─────────────────────────────────────────────────────────────┐
│ │
│ CORRECTITUD: │
│ □ ¿Hace lo que se supone? │
│ □ ¿Edge cases manejados? │
│ □ ¿Condiciones de error cubiertas? │
│ │
│ DISEÑO: │
│ □ ¿El enfoque es apropiado? │
│ □ ¿Encaja en la arquitectura? │
│ □ ¿Es mantenible? │
│ │
│ LEGIBILIDAD: │
│ □ ¿Nombres claros? │
│ □ ¿Comentarios apropiados? │
│ □ ¿Complejidad razonable? │
│ │
│ TESTING: │
│ □ ¿Tests incluidos? │
│ □ ¿Cobertura de tests adecuada? │
│ □ ¿Edge cases testeados? │
│ │
│ SEGURIDAD: │
│ □ ¿Datos sensibles no expuestos? │
│ □ ¿Validación de input presente? │
│ □ ¿Auth/autorización correcta? │
└─────────────────────────────────────────────────────────────┘
Feedback Constructivo
COMENTARIOS DE REVIEW EFECTIVOS:
┌───────────────────────────────┬─────────────────────────────┐
│ EN VEZ DE: │ INTENTA: │
├───────────────────────────────┼─────────────────────────────┤
│ "Esto está mal" │ "Esto podría causar X. │
│ │ ¿Consideras Y en su lugar?"│
├───────────────────────────────┼─────────────────────────────┤
│ "Mal nombre" │ "¿Qué tal `userCount` │
│ │ para más claridad?" │
├───────────────────────────────┼─────────────────────────────┤
│ "¿Por qué hiciste esto?" │ "Tengo curiosidad sobre el │
│ │ razonamiento aquí - ¿puedes│
│ │ explicar?" │
└───────────────────────────────┴─────────────────────────────┘
PREFIJOS DE COMENTARIOS:
[nit] - Preferencia menor de estilo, opcional
[sugerencia] - Considera este enfoque
[pregunta] - Necesito aclaración
[bloqueador] - Debe arreglarse antes de merge
Flujo de Trabajo de Review
Integración con GitScrum
CICLO DE VIDA PR → TAREA:
┌─────────────────────────────────────────────────────────────┐
│ │
│ ACTUALIZACIONES DE ESTADO AUTOMÁTICAS: │
│ │
│ PR Creado → Tarea se mueve a "En Revisión" │
│ PR Aprobado → Tarea se mueve a "Aprobado" │
│ PR Merged → Tarea se mueve a "Hecho" │
│ PR Cerrado → Tarea vuelve a "En Progreso" │
│ │
│ VISIBILIDAD EN GITSCRUM: │
│ ┌─────────────────────────────────────────────────────────┐│
│ │ Tarea: Implementar login OAuth ││
│ │ ││
│ │ PR: #456 - feat: add oauth login ││
│ │ Estado: ✅ Aprobado (2 aprobaciones) ││
│ │ Checks: ✅ CI pasando ││
│ │ Tiempo en review: 4 horas ││
│ │ ││
│ │ [Ver PR] [Mergear] ││
│ └─────────────────────────────────────────────────────────┘│
└─────────────────────────────────────────────────────────────┘
Métricas de Review
Dashboard de Reviews
MÉTRICAS DE CODE REVIEW:
┌─────────────────────────────────────────────────────────────┐
│ │
│ ESTA SEMANA: │
│ │
│ PRs Abiertos: 12 │ PRs Merged: 28 │ Promedio: 4h │
│ │
│ TIEMPO DE REVIEW POR REVISOR: │
│ ├── @alex: 3.2h promedio (8 reviews) │
│ ├── @jordan: 5.1h promedio (6 reviews) │
│ ├── @sam: 2.8h promedio (10 reviews) │
│ └── @pat: 6.5h promedio (4 reviews) │
│ │
│ ALERTAS: │
│ ⚠️ 2 PRs esperando review > 24 horas │
│ ⚠️ @jordan tiene carga de review alta │
│ │
│ TENDENCIA: │
│ Tiempo a merge: ▼ mejorando (-15% vs mes anterior) │
│ │
└─────────────────────────────────────────────────────────────┘