Cuándo hacer la Revisión de Código

15

Recientemente hemos pasado a un proceso de scrum y estamos trabajando en tareas e historias de usuarios dentro de los sprints. Nos gustaría hacer revisiones de código con frecuencia para hacerlas menos desalentadoras. Estamos pensando en realizarlos en una historia de usuario, pero no estamos seguros de cómo derivar nuestro código para dar cuenta de esto.

Estamos utilizando VS y TFS 2010 y somos un equipo de 6.

Actualmente estamos bifurcando para características, pero estamos trabajando en cambiar a bifurcar para scrum.

Actualmente no utilizamos conjuntos de estantes y realmente no queremos implementar si hay otras técnicas disponibles.

¿Cómo recomienda que implementemos la revisión de código por historia de usuario?

    
pregunta mcass20 08.03.2011 - 20:14
fuente

5 respuestas

3

Depende de la naturaleza de las historias de usuario.

Puede ser efectivo crear una rama para cada historia de usuario, el progreso en diferentes historias es visible, se puede transmitir si es necesario, si las historias no se completan en el sprint, entonces el progreso puede permanecer en la rama durante el siguiente sprint. Las revisiones finales se pueden realizar al final de una historia de usuario en la rama de uso de la historia y combinarlas si el código es estándar.

Para trabajar de la manera en que las historias deben estar finamente integradas para evitar tareas de fusión inmanejables al final de un sprint. Las historias pequeñas permitirán una actualización constante de la rama de desarrollo a través del sprint que los desarrolladores que trabajan en otras historias de usuario necesitan extraer constantemente (VCM básico).

Esto crea sobrecargas de procesos al tener que crear y fusionar sucursales constantemente, lo que en algunos casos se puede resolver con scripts de automatización, pero el equipo aún necesita sentirse muy cómodo con el VCS.

Al final de un sprint, fusionas tu rama de desarrollo en integración / producción, etc.

También he trabajado en equipos en los que todos trabajan en una rama de desarrollo, al completar una historia de usuario, el código se envía a esa rama para su revisión y prueba, y si alguien empuja algo que rompe la versión de desarrollo, tiene que obtener el equipo. cervezas en.

    
respondido por el whatsthebeef 08.03.2011 - 20:35
fuente
13

La forma más efectiva de revisar el código es levantarse, encontrar a alguien y pedirle que venga y discuta el código que acaba de desarrollar.

No use una herramienta a menos que no pueda encontrar a alguien que revise su código localmente.

Puedes evitar las revisiones de código por completo.

    
respondido por el Ryan Cromwell 08.03.2011 - 22:32
fuente
4

¿Todos en el equipo son locales? Si es así, solo pídale a alguien que venga y eche un vistazo antes de verificar el código. ¿No es local? Encienda su programa favorito de pantalla compartida y llame a alguien. Yo personalmente hago esto a menudo. A veces lo hago solo para decir "¡Eh, mira lo que hice!"

Prefiero este estilo de revisiones de código ad-hoc al estilo donde alguien está de pie y presentando su código al equipo. Las revisiones ad-hoc pueden proporcionarle muchos (¿todos?) Los beneficios del emparejamiento sin la incomodidad. Además, es más probable que su "revisor" haga preguntas y sugiera mejoras en un entorno informal y personal.

    
respondido por el jgrim 09.03.2011 - 00:54
fuente
1

Creo que la revisión de código no es una parte formal de SCRUM, pero las revisiones son una táctica independiente para lograr calidad y mejorar sus proyectos / equipo.

Por lo tanto, usaría SCRUM (u otra metodología de desarrollo ágil) para garantizar / mejorar la calidad del PROYECTO y mantener el calendario. Además, una buena táctica es hacer la revisión del producto (no el código) de forma independiente de sus tareas de control de calidad / pruebas normales. Si esta actividad se puede hacer frente a su equipo / socios / clientes / audiencia, será mejor.

Debe usar revisiones de código (u otras) específicas principalmente para mejorar su EQUIPO, esperando resultados a medio / largo plazo. Esto afectará sus PROYECTOS, pero a largo plazo como producto de la mejora de su EQUIPO.

Entonces, para responder a tu pregunta, creo que estás tratando de presionar demasiado de SCRUM, y deberías considerar las revisiones solo como están.

    
respondido por el Ron-Damon 08.03.2011 - 22:07
fuente
0

¿No es obvio hacer revisiones de código antes de registrar su código?

TFS no funciona como GIT, por lo que cada vez que marque el código en una rama o troncal, estará disponible para todos.

Esto significa que la revisión debe realizarse en el momento del registro, para que los cambios erróneos no se propaguen a la copia de trabajo de todos.

    
respondido por el Byron Whitlock 08.03.2011 - 20:16
fuente

Lea otras preguntas en las etiquetas