Práctica donde otros desarrolladores revisan el código escrito por un compañero.
Identifica errores, mejora la calidad del código y asegura el cumplimiento de las buenas prácticas.
// Checklist de Code Review // 1. Funcionalidad // ✅ ¿El código hace lo que debería? // ✅ ¿Maneja casos edge correctamente? // ✅ ¿Los tests cubren la funcionalidad? // 2. Legibilidad // ✅ ¿Los nombres son descriptivos? // ✅ ¿El código es fácil de entender? // ✅ ¿Hay comentarios donde son necesarios? // 3. Seguridad // ✅ ¿Se validan las entradas del usuario? // ✅ ¿Se manejan los errores correctamente? // ✅ ¿No hay datos sensibles hardcodeados? // 4. Rendimiento // ✅ ¿Hay queries N+1? // ✅ ¿Se usa caching donde corresponde? // ✅ ¿Los componentes re-renderizan innecesariamente? // Ejemplo de comentario de review // ❌ "Esto está mal" // ✅ "Considera usar useMemo aquí para evitar // recalcular en cada render. Ejemplo: // const valor = useMemo(() => calcular(data), [data]);" // Herramientas: // - GitHub Pull Requests // - GitLab Merge Requests // - Reviewable, Crucible
