5 min lectura • Guide 650 of 877
Optimización del Proceso de Code Review
Los code reviews son cruciales para la calidad del código y compartir conocimiento, pero procesos deficientes crean cuellos de botella frustrantes. GitScrum ayuda a rastrear el estado de revisión, identificar backlogs de review y medir tiempos de respuesta para ayudar a los equipos a optimizar sus flujos de trabajo de revisión.
Diseño del Proceso de Review
Etapas del Flujo de Trabajo
PIPELINE DE CODE REVIEW:
┌─────────────────────────────────────────────────────────────┐
│ │
│ PR CREADO → LISTO PARA REVIEW → EN REVISIÓN → APROBADO │
│ │ │ │ │
│ ▼ ▼ ▼ │
│ [Esperando] [Activo] [Merge] │
│ Objetivo: <4h Objetivo:<24h Objetivo:<2h │
│ │
├─────────────────────────────────────────────────────────────┤
│ │
│ ESTADO ACTUAL: │
│ Listo para Review: 3 PRs (más antiguo: 2h) ✅ Saludable │
│ En Revisión: 2 PRs ✅ Saludable │
│ Aprobado: 1 PR (esperando merge) ✅ Saludable │
│ │
│ ALERTAS: │
│ ⚠️ PR #234 esperando 6h (SLA excedido) │
│ │
└─────────────────────────────────────────────────────────────┘
SLAs de Review
EXPECTATIVAS DE REVIEW:
┌─────────────────────────────────────────────────────────────┐
│ │
│ PRIORIDAD │ PRIMERA RESP. │ COMPLETADO │ ESCALACIÓN │
│─────────────┼───────────────┼────────────┼─────────────────│
│ Crítico │ 2 horas │ 4 horas │ Después 3h │
│ Alto │ 4 horas │ 24 horas │ Después 6h │
│ Normal │ 24 horas │ 48 horas │ Después 36h │
│ Bajo │ 48 horas │ 72 horas │ Después 60h │
│ │
│ TIPOS DE RESPUESTA: │
│ • Aprobar: Listo para merge │
│ • Comentar: Preguntas/sugerencias (no bloqueante) │
│ • Solicitar Cambios: Debe abordar antes de merge │
│ │
│ CUÁNDO SOLICITAR CAMBIOS: │
│ • Vulnerabilidades de seguridad │
│ • Breaking changes sin migración │
│ • Tests faltantes para rutas críticas │
│ • Bugs claros en lógica │
└─────────────────────────────────────────────────────────────┘
Mejores Prácticas de Pull Request
Guías de Tamaño de PR
TAMAÑO ÓPTIMO DE PR:
┌─────────────────────────────────────────────────────────────┐
│ │
│ LÍNEAS │ TIEMPO REV. │ CALIDAD │ RECOMENDACIÓN │
│──────────┼──────────────┼────────────┼────────────────────│
│ <100 │ 15-30 min │ Excelente │ ✅ Ideal │
│ 100-400 │ 30-60 min │ Buena │ ✅ Aceptable │
│ 400-800 │ 60-90 min │ Declinando │ ⚠️ Considerar split│
│ 800+ │ 90+ min │ Pobre │ ❌ Muy grande │
│ │
│ ESTRATEGIAS PARA PRs MÁS PEQUEÑOS: │
│ │
│ 1. Feature flags │
│ Enviar features incompletas detrás de flags │
│ │
│ 2. PRs apilados │
│ PR 1: Modelo de datos → PR 2: API → PR 3: UI │
│ │
│ 3. Extraer refactoring │
│ Refactorizar primero, feature segundo │
│ │
│ 4. Slices verticales │
│ Enviar feature delgada end-to-end │
└─────────────────────────────────────────────────────────────┘
Template de Descripción de PR
TEMPLATE DE PR:
┌─────────────────────────────────────────────────────────────┐
│ ## Qué │
│ [Descripción de una línea del cambio] │
│ │
│ ## Por qué │
│ [Contexto: problema siendo resuelto, link a tarea] │
│ │
│ ## Cómo │
│ [Enfoque técnico breve, decisiones clave] │
│ │
│ ## Testing │
│ - [ ] Tests unitarios agregados/actualizados │
│ - [ ] Testing manual completado │
│ - [ ] Edge cases considerados │
│ │
│ ## Screenshots (si cambio de UI) │
│ [Screenshots antes/después] │
│ │
│ ## Foco del Review │
│ [En qué te gustaría que los revisores se enfoquen] │
│ │
│ ## Checklist │
│ - [ ] Auto-revisado │
│ - [ ] Tests pasan │
│ - [ ] Documentación actualizada │
└─────────────────────────────────────────────────────────────┘
Métricas de Review
Dashboard de Revisiones
MÉTRICAS DE CODE REVIEW:
┌─────────────────────────────────────────────────────────────┐
│ │
│ ESTA SEMANA: │
│ │
│ Tiempo promedio primera respuesta: 3.2 horas ✅ │
│ Tiempo promedio completado: 18 horas ✅ │
│ PRs merged: 28 │
│ PRs en backlog: 5 │
│ │
│ POR REVISOR: │
│ ├── @alex: 8 reviews, 2.5h promedio │
│ ├── @jordan: 6 reviews, 4.1h promedio │
│ ├── @sam: 10 reviews, 2.0h promedio │
│ └── @pat: 4 reviews, 6.0h promedio │
│ │
│ TENDENCIA: │
│ Tiempo a merge: ▼ -15% vs mes anterior │
│ │
└─────────────────────────────────────────────────────────────┘