¿Deberíamos usar Rechazar o Esperar al autor en TFS cuando hay cosas que arreglar?

12

En TFS, cuando colocamos comentarios para arreglar las cosas en una solicitud de extracción antes de aceptarla, ¿deberíamos marcarla como Rechazar o Esperar al autor? ¿Cual es mejor?

ingrese la descripción de la imagen aquí

sashoalm
fuente

Respuestas:

14

Según Microsoft en el código de Revisión con solicitudes de extracción: Votar sobre los cambios el propósito sugerido de cada clase de aprobación es:

Aprobar con sugerencias : aceptar la solicitud de extracción, pero proporcionar sugerencias opcionales para mejorar el código.

Esperando al autor : no apruebe los cambios y pídale al autor que revise sus comentarios. El autor debe informarle cuándo debe volver a revisar el código después de que haya abordado sus inquietudes.

Rechazado : los cambios no son aceptables. Si vota de esta manera, debe dejar un comentario en la solicitud de extracción que detalle por qué se rechazaron los cambios.

Por lo tanto, quiero Waiting for Authordecir que crees que el autor se equivocó en su enfoque de la solución, pero que su código es canjeable si toman en serio tus comentarios.

Y Rejectedsignifica que de ninguna manera estás aceptando un cambio como este, sin importar qué tan bien esté escrito el código.

La pregunta que debe hacerse es si la idea de su grupo del propósito sugerido coincide con la idea de Microsoft.

Peter M
fuente
1
Acerca de esto: "El autor debe informarle cuándo debe volver a revisar el código después de que haya abordado sus inquietudes". - ¿Existe un mecanismo para que el autor avise a los revisores que el RP está listo para una nueva revisión?
Robert Sim
@RobertSim: utilizamos la opción 'Ping' en Revisores para enviar un mensaje solicitando una nueva revisión.
killercowuk