Tradicionalmente, realizamos la revisión del código antes de confirmar, tuve una discusión con mi colega hoy, quien prefirió la revisión del código después de la confirmación.
Primero, aquí hay algunos antecedentes,
- Tenemos algunos desarrolladores con experiencia y también tenemos nuevas contrataciones con una experiencia de programación casi nula.
- Nos gustaría realizar iteraciones rápidas y cortas para lanzar nuestro producto.
- Todos los miembros del equipo se encuentran en el mismo sitio.
Las ventajas de la revisión de código antes de confirmar que he aprendido:
- Nuevas contrataciones de mentores
- Intente evitar errores, fallas, malos diseños al principio del ciclo de desarrollo
- Aprende de otros
- Copia de seguridad del conocimiento si alguien se cierra
Pero también he tenido algunas malas experiencias:
- Baja eficiencia, algunos cambios pueden revisarse en días
- Es difícil equilibrar la velocidad y la calidad, especialmente para los novatos
- Un miembro del equipo sintió desconfianza
En cuanto a la revisión posterior a la confirmación, sé poco sobre esto, pero lo que más me preocupa es el riesgo de perder el control debido a la falta de revisión. ¿Alguna opinión?
ACTUALIZAR :
- Estamos usando Perforce para VCS
- Codificamos y confirmamos en las mismas ramas (ramas troncales o de corrección de errores)
- Para mejorar la eficiencia, hemos tratado de dividir el código en pequeños cambios. También hemos probado algunas revisiones de diálogos en vivo, pero no todos siguieron la regla. Sin embargo, este es otro problema.