Revisión de código que mejora en vez de retrasar

Tamaño

Una solicitud de fusión de doscientas líneas recibe comentarios reales; una de dos mil recibe 'se ve bien'. Divídela. Si no puedes dividirla, escribe en la descripción una ruta de lectura recomendada.

Sobre qué comentar

Corrección, casos límite, seguridad, rendimiento en los puntos sensibles, y claridad para quien lo lea dentro de un año. Estilo, espacios y el orden de las importaciones: para una herramienta automática, no para una persona.

Formula los comentarios como una pregunta o una sugerencia, y marca qué bloquea y qué es solo una opinión. 'Bloqueante: esto permite el acceso al recurso de otro usuario' es más claro que una insinuación amable.

Tiempo de respuesta

Una revisión que espera dos días detiene a las personas y produce ramas largas. Fija una norma —por ejemplo, responder en medio día— y trátala como cualquier compromiso del equipo.