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

Estamos comenzando a usar 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 realiza el solicitante estos cambios y los muestra?

Así que 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 lo necesario cambio.

¿Cómo muestra ahora la Persona 1 estos cambios a la Persona 2? Puede agregar comentarios y enviarlos, pero los archivos no cambian. Asumo 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 pierdes el historial de tu conversación de por qué estabas haciendo los cambios.

Hay un montón de sitios web que muestran cómo utilice 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 junto a archivos?

Author: Machavity, 2013-08-20
Source

4 answers

Así que 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. La 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 shelveset "
  2. Desde el panel" Detalles del conjunto de estanterías " resalte y copie el nombre del conjunto de estanterías
  3. Vaya al panel "Cambios pendientes", haga clic en" Archivar " y pegue el nombre del conjunto de estantes
  4. Presione el botón Sí en el diálogo de verificación reemplazar conjunto de estanterías
  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 me parece que ayuda a aclarar las cosas.

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

introduzca la descripción de la imagen aquí

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

introduzca la descripción de la imagen aquí

3) Vaya al panel "Cambios pendientes", haga clic en "Shelve" y pegue el nombre del conjunto de estantes, por ejemplo:

introduzca la descripción de la imagen aquí

4) Presione el botón Sí en el conjunto de estanterías reemplazar verificación diálogo:

introduzca la descripción de la imagen aquí

 130
Author: Chad Dienhart,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2014-10-29 19:20:20

Creo que el procedimiento correcto es que la Persona 1 haga los cambios y solicite otra revisión. Cuando su código necesita trabajar, eso significa que lo cambiará, por lo que querrá tener la versión anterior para mirar hacia atrás para la comparación. Usted todavía tiene la vieja revisión en la historia después de que se cierra si desea mirar por encima de los comentarios. Actualmente estamos en el proceso de optimizar nuestro proceso de revisión de código en mi lugar de trabajo.

 5
Author: Paddy,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2013-08-20 15:19:35

Sé que esta pregunta es antigua, pero todavía no está apoyada como lo señalan otros posters. 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 uservoice muy antigua para permitir la actualización de una Revisión de Código que debería resolver elegantemente este problema haciendo que las Revisiones de Código tengan iteraciones.

 1
Author: julealgon,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2015-05-29 21:19:04

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 revisiones basadas en conjuntos de cambios, pero también funciona para revisiones basadas en estantes.

  1. Crear tasa1
  2. Antes de registrarse en changeset1, agregue task1 como un elemento de trabajo relacionado
  3. Compruebe los cambios con un elemento de trabajo relacionado y solicite una revisión de este conjunto de cambios
  4. Crear tasa2
  5. Antes comprobación en changeset2 añadir ambas tareas como un elemento de trabajo relacionado.
  6. Verifique 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 el revisor busca task1, ve el conjunto de cambios 1 y la solicitud de revisión con sus comentarios. Así que no perderás el historial de conversaciones.

 0
Author: Ville Pelho,
Warning: date(): Invalid date.timezone value 'Europe/Kyiv', we selected the timezone 'UTC' for now. in /var/www/agent_stack/data/www/ajaxhispano.com/template/agent.layouts/content.php on line 61
2013-10-08 07:30:45