Estamos comenzando a usar la funcionalidad de revisión de código integrada en la vista previa de VS 2012 y VS 2013. Solicitar la revisión y agregar comentarios parece bastante sencillo. Si alguien agrega comentarios solicitando que se cambie el código, ¿cómo hace el solicitante para hacer estos cambios y mostrarlos?
Entonces el proceso fluiría así:
- La persona 1 solicita una revisión del código.
- La persona 2 agrega comentarios y selecciona "Necesita trabajo".
- La persona 1 hace los cambios necesarios.
¿Cómo muestra ahora la Persona 1 estos cambios a la Persona 2? Puede agregar comentarios y enviarlos, pero los archivos no cambian. Supongo que los archivos son del conjunto de cambios creado cuando se solicitó la revisión original. ¿Debería la Persona 1 cerrar esta revisión y solicitar una segunda revisión?
Crear una segunda revisión no parece óptimo, porque pierde el historial de su conversación de por qué estaba haciendo los cambios.
Hay un montón de sitios web que muestran cómo usar la funcionalidad básica de las revisiones de código, pero ¿hay algún sitio que muestre las mejores prácticas? Por ejemplo, ¿quién debería hacer clic en las casillas de verificación junto a los archivos?
Respuestas:
Entonces el proceso fluiría así:
Estos son los pasos necesarios para actualizar el conjunto de estanterías asociado con la revisión.
He incluido algunas capturas de pantalla porque creo que ayuda a aclarar las cosas.
1) En el panel "Revisión de código", seleccione el enlace "ver conjunto de estantes" como se muestra aquí:
2) Desde el panel "Detalles de Shelveset" resalte y copie el nombre de Shelveset como se muestra aquí:
3) Navegue hasta el panel "Cambios pendientes", haga clic en "Shelve" y pegue el nombre del conjunto de shelve, por ejemplo:
4) Presione el botón Sí en el cuadro de diálogo de verificación de reemplazo de estantería:
fuente
Creo que el procedimiento correcto es que la Persona 1 realice los cambios y solicite otra revisión. Cuando su código necesita trabajo, eso significa que lo cambiará, por lo que querrá tener la versión anterior para comparar. Todavía tiene la revisión anterior en el historial después de que se cierra si desea revisar los comentarios. Actualmente estamos en el proceso de optimizar nuestro proceso de revisión de código en mi lugar de trabajo.
fuente
Sé que esta pregunta es antigua, pero aún no es compatible, como lo señalan otros carteles. La solución propuesta por chad funcionará para algunas cosas pero tendrá un comportamiento extraño para otras.
Recientemente, el equipo de TFS comenzó las etapas de planificación para una solución a una solicitud de voz de usuario muy antigua para permitir la actualización de una Revisión de Código que debería resolver este problema con elegancia al hacer que las Revisiones de Código tengan iteraciones.
fuente
Tienes que hacer esto con dos reseñas diferentes. Pero también hay una manera de mantener el historial con la segunda revisión. Todo lo que necesitas son tareas.
Este flujo de trabajo se describe para revisiones basadas en conjuntos de cambios, pero también funciona para revisiones basadas en estanterías.
Ahora, en la segunda solicitud de revisión, el revisor puede buscar tareas relacionadas y, si revisa la tarea1, ve el conjunto de cambios1 y la solicitud de revisión con sus comentarios. Entonces no perderás el historial de conversación.
fuente