¿Qué es una code review?
Opciones
- Contar cuántas líneas de código ha escrito alguien
- Otra persona desarrolladora lee tus cambios antes de que se fusionen (Opción correcta)
- El compilador comprobando la sintaxis de todos los archivos
- Una demo del producto terminado para un cliente
Respuesta
Otra persona desarrolladora lee tus cambios antes de que se fusionen, para detectar bugs, mejorar la calidad y compartir conocimiento
Por qué
En la mayoría de los equipos no puedes meter código en la rama principal por tu cuenta. Abres un pull request y tus compañeros lo comentan.
Una revisión busca bugs, nombres poco claros, tests que faltan, problemas de seguridad y coherencia con el resto del código.
Las buenas revisiones son educadas y concretas, y enseñan algo a ambos lados.
Recuerda
Code review = tus compañeros revisan tu cambio antes de fusionarlo.