Revision de codigo

Una revision de codigo apresurada deja pasar justo lo que mas cuesta arreglar despues: un caso limite sin cubrir, un fallo de seguridad discreto o un nombre confuso que otro tendra que descifrar dentro de seis meses. Revisar bien no es aprobar rapido, es proteger a quien vendra luego a tocar ese mismo archivo.

Esta plantilla ordena la revision de un pull request en pasos concretos para que ninguno se te escape cuando tienes varios cambios esperando. La idea no es alargar el proceso, es recorrer siempre los mismos puntos para que la calidad no dependa del dia que tengas ni de las prisas del que pide la revision.

Que revisar en cada pull request

Estos son los puntos que trae la plantilla, pensados para mirar el cambio de fuera hacia dentro:

• Entender el objetivo del cambio antes de leer una sola linea
• Comprobar que la logica es correcta y hace lo que promete
• Revisar el manejo de errores y que no se traguen excepciones en silencio
• Buscar casos limite sin cubrir, como listas vacias o valores nulos
• Verificar que hay pruebas suficientes para lo que se toca
• Revisar nombres y legibilidad para que se entienda sin explicaciones
• Detectar codigo duplicado que convendria extraer
• Valorar el impacto en el rendimiento en las rutas mas usadas
• Revisar posibles fallos de seguridad en entradas y permisos
• Dejar comentarios claros y utiles, no solo senalar lo que esta mal

Como funciona en Foco

Al importarla, Foco crea el trabajo "Revision" con estos puntos en la vista Lista, para que los recorras en orden en cada pull request y marques cada uno segun lo compruebas. Como la revision se repite en cada cambio, puedes dejarla como recurrente y tener la lista fresca cada vez.

Si compartes las revisiones con otra persona, asignas cada punto a quien lo mira, marcas como importante lo que no puede quedar sin revisar y dejas por voz un apunte rapido sobre algo que quieres discutir sin abandonar la tarea.