Paso 2 de la solicitud de extracción: revisión
Escrito para la antigua interfaz de usuario
Este artículo fue escrito para la antigua interfaz de usuario. Las instrucciones todavía se pueden usar, pero las capturas de pantalla están desactualizadas. En algunos casos, los elementos mencionados en las instrucciones se han movido o cambiado. Si no puede encontrar un elemento, consulte el ¿Dónde quedó? artículo.
Resumen
Este artículo explica cómo completar el proceso de revisión de una solicitud de extracción, que involucra tanto al creador como a los revisores asignados. Realizar revisiones exhaustivas es vital para mantener la calidad del código.
Mejores prácticas
Agrega siempre al menos un revisor para garantizar la responsabilidad.
Usa el campo de descripción para resumir los cambios clave para los revisores.
Haz la fusión solo cuando todos los conflictos estén resueltos y las pruebas hayan pasado.
Usa Solicitar cambios en lugar de simplemente no aprobar, ya que mantiene un flujo de trabajo claro.
Evita la autoevaluación y fusionar sin un revisor, a menos que sea una corrección crítica de tiempo.
Pasos de Pull Request
Explora nuestra serie de artículos que te guían a través de cada paso del proceso de Pull Request. Puedes leerlos en orden o seleccionar un artículo específico para obtener orientación detallada sobre una etapa en particular.
Pestaña de resumen de la solicitud de extracción
Los detalles de una solicitud de extracción ayudan a los revisores y colaboradores a obtener contexto sobre la solicitud. La Resumen pestaña tiene dos secciones para revisar: Detalles de la solicitud de extracción y Revisores de la solicitud de extracción.
Detalles de la solicitud de extracción
Estado
Estado actual de la solicitud de extracción (Creada, Revisión solicitada, Cambios solicitados, y Aprobada).
Rama de destino
La rama donde se fusionarán los cambios.
Rama de origen
La rama donde se originan los cambios.
Creado por
Usuario que creó la solicitud de extracción.
Fecha de creación
Cuándo se creó por primera vez la solicitud de extracción.
Fecha de actualización
Cuándo ocurrió la última actividad o cambio en la solicitud de extracción.
Descripción
Resumen de los cambios de esta solicitud de extracción. Escrito por el creador de la solicitud de extracción.
Revisores de la solicitud de extracción
La Revisores sección muestra a los usuarios asignados para revisar esta solicitud de extracción. El estado de cada revisor se indica mediante un icono que ilustra el estado de la revisión (revisión pendiente, cambios solicitados o aprobado). Consulta las instrucciones a continuación para obtener más detalles sobre el proceso de revisión.
Revisar la solicitud de extracción
Espera la comprobación de conflictos de fusión
Cuando se crea una solicitud de extracción, primero se comprueban los conflictos de fusión, lo que se indica con un banner gris.
Espera a que el banner cambie a uno de los siguientes:
Verde: Esta rama se puede fusionar
Naranja: Esta rama tiene conflictos que deben resolverse
Si la rama se puede fusionar, puedes continuar revisando la solicitud de extracción siguiendo las instrucciones de este artículo.
Si la rama tiene conflictos, debes resolverlos antes de continuar con la revisión de esta solicitud de extracción. Consulta el siguiente artículo para aprender cómo resolver conflictos.
(Opcional) Agregar revisores
Agregar revisores garantiza la calidad del código, ayuda a identificar conflictos tempranamente y hace cumplir los procesos de aprobación del equipo. Todos los revisores deben hacer clic en Aprobar antes de que la solicitud de extracción pueda fusionarse.
Autoevaluación
Si no se agregan revisores, puedes revisar por tu cuenta los cambios de tu solicitud de extracción. No se requiere revisión ni aprobación formal. Después de revisar, puedes fusionar sin aprobación adicional.
En la Revisores sección, haz clic en el icono de lápiz .
Haz clic en Agregar para cada usuario que quieras que revise esta solicitud de extracción.
Haz clic en Guardar.
Los revisores agregados aparecerán en la Revisores sección, y el estado de la solicitud de extracción cambiará a Revisión solicitada.
Revisar cambios
Para revisar la solicitud de extracción, cada revisor debe iniciar sesión en Cloud DevOps y seguir estos pasos:
Autoevaluación
Si estás haciendo una autoevaluación, aún debes revisar cada componente antes de fusionar.
Haz clic en Solicitudes de extracción desde el menú de la barra lateral izquierda.
Haz clic en la solicitud de extracción que quieras revisar.
Haz clic en la pestaña Componentes modificados .
Haz clic en un componente para revisarlo.
Todos los componentes deben revisarse.
Al ver un componente, el código fuente de Destino y Origen se muestra lado a lado con las diferencias resaltadas.
Rojo: Líneas que se eliminarán.
Verde: Líneas que se agregarán.
Repite el proceso de revisión para cada componente.
(Opcional) Solicitar cambios
Al revisar, usa Solicitar cambios para indicar que se necesitan correcciones antes de la aprobación. Este estado solicita al creador de la solicitud de extracción que actualice la rama de origen con los cambios necesarios. Como la solicitud no especifica qué debe cambiarse, comunica directamente a tu equipo los cambios específicos requeridos.
Haz clic en Solicitar cambios.
Después de solicitar cambios, el estado de la solicitud de extracción cambiará a Cambios solicitados.
Reenviar para revisión
Después de hacer los cambios solicitados, vuelve a enviar la solicitud de extracción para que la revise el revisor que los solicitó.
Haz clic en la solicitud de extracción.
Haz clic en la pestaña Actualizar icono junto al revisor que solicitó la revisión.
Aprobar
Una vez que un revisor haya completado la revisión de todos los componentes y de cualquier solicitud de cambio completada, debe aprobar la solicitud de extracción antes de que pueda fusionarse.
La solicitud de extracción no se aprobará hasta que todos los revisores hagan clic en Aprobar.
Autoevaluación
Cuando haces una autoevaluación, no puedes Aprobar la solicitud de extracción. Pero como no hay revisores, la aprobación no es necesaria antes de fusionar.
Haz clic en Aprobar.
Después de que todos los revisores aprueben, el estado de la solicitud de extracción cambiará a Aprobada, y se podrá fusionar.
Última actualización
¿Te fue útil?