Revisiones de código TFS - Mostrar archivos actualizados en respuesta a comentarios

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í:

1. La persona 1 solicita una revisión del código.
2. La persona 2 agrega comentarios y selecciona "Necesita trabajo".
3. 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?

Entonces el proceso fluiría así:

1. La persona 1 solicita una revisión del código.
2. La persona 2 agrega comentarios y selecciona "Necesita trabajo".
3. La persona 1 hace los cambios necesarios.
4. Persona 1 Actualiza el conjunto de estanterías asociado con la revisión del código
5. La persona 1 agrega comentarios para continuar la discusión
6. Repita los pasos 2 - 5 hasta que sea aceptado

Estos son los pasos necesarios para actualizar el conjunto de estanterías asociado con la revisión.

1. Desde el panel "Revisión de código", seleccione el enlace "ver conjunto de anaqueles"
2. Desde el panel "Detalles de la estantería", resalte y copie el nombre de la estantería
3. Navegue hasta el panel "Cambios pendientes", haga clic en "Shelve" y pegue el nombre del conjunto de shelve
4. Presione el botón Sí en el cuadro de diálogo Reemplazar verificación
5. Ahora el revisor puede ver los archivos actualizados y la discusión de revisión puede continuar

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:

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.

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.

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.

1. Crear tarea1
2. Antes de registrar changeset1 agregue task1 como elemento de trabajo relacionado
3. Registre los cambios con un elemento de trabajo relacionado y solicite una revisión de este conjunto de cambios
4. Crear tarea2
5. Antes de registrar changeset2, agregue ambas tareas como un elemento de trabajo relacionado.
6. Registre los cambios con dos elementos de trabajo relacionados y solicite una revisión de este conjunto de cambios

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.