c# - generate - Revisiones de código TFS: muestra los archivos actualizados en respuesta a los comentarios

params comments c#

Question

Estamos comenzando a utilizar la funcionalidad de revisión de código incorporada en VS 2012 y VS 2013 preview. 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 estos cambios y los muestra?

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 la persona 1 ahora muestra 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 una gran cantidad de sitios web que muestran cómo usar la funcionalidad básica de las revisiones de código, pero ¿hay sitios que muestren las mejores prácticas? Por ejemplo, ¿quién debería hacer clic en las casillas de verificación al lado de los archivos?

Answer 1

Creo que el procedimiento correcto es que la persona 1 haga los cambios y solicite otra revisión. Cuando su código necesita funcionar, significa que lo va a cambiar, por lo que querrá tener la versión anterior para mirar atrás y 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.

Answer 2

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 estantes 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 estantes asociado con la revisión.

1. Desde el panel "Revisión de código", seleccione el enlace "ver estante de biblioteca"
2. En el panel "Detalles de Shelveset", resalte y copie el nombre del archivador
3. Vaya al panel "Cambios pendientes", haga clic en "Guardar" y pegue el nombre del conjunto de armarios
4. Presione el botón Sí en el conjunto de armarios para reemplazar el diálogo de 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, ya que creo que ayuda a aclarar las cosas.

1) Desde el panel "Revisión de código", seleccione el enlace "ver estantería" como se muestra aquí:

2) Desde el panel "Detalles de Shelveset", resalte y copie el nombre del archivador como se muestra aquí:

3) Navegue hasta el panel "Cambios pendientes", haga clic en "Guardar" y pegue el nombre del conjunto de armarios, por ejemplo:

4) Presione el botón Sí en el diálogo de verificación de reemplazo del conjunto de armarios:

Answer 3

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 otros.

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 elegantemente haciendo revisiones de código con iteraciones.

Answer 4

Tienes que hacer esto con dos revisiones diferentes. Pero también hay una manera de mantener la historia con la segunda revisión. Todo lo que necesitas son tareas.

Este flujo de trabajo se describe para las revisiones basadas en conjuntos de cambios, pero también funciona para las revisiones basadas en estantes.

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

Ahora, en la segunda solicitud de revisión, el revisor puede buscar tareas relacionadas y si el revisor busca la tarea 1, ve el conjunto de cambios1 y la solicitud de revisión con sus comentarios. Entonces no perderás el historial de conversación.