Solicitud de extracción Paso 2: Revisión
Resumen
ResumenEste artículo explica cómo completar el proceso de Revisión para una Solicitud de Extracción (Pull Request), 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
Siempre agrega al menos un revisor para responsabilidad.
Usa el campo de descripción para resumir los cambios clave para los revisores.
Solo fusiona cuando todos los conflictos estén resueltos y las pruebas hayan pasado.
Usar Solicitar cambios en lugar de simplemente no aprobar, ya que mantiene un flujo de trabajo claro.
Evita la auto-revisión y la fusión sin un revisor a menos que sea una corrección crítica por tiempo.
Pestaña Resumen de la Solicitud de Extracción
Los detalles de una Solicitud de Extracción ayudan a los revisores y contribuyentes 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 destino
La rama donde se fusionarán los cambios.
Rama origen
La rama de donde se originan los cambios.
Creado por
Usuario que creó la Solicitud de Extracción.
Fecha de creación
Cuando la Solicitud de Extracción fue creada por primera vez.
Fecha de actualización
Cuando ocurrió la última actividad o cambio en la Solicitud de Extracción.
Descripción
Resumen de los cambios para esta Solicitud de Extracción. Escrito por el creador de la Solicitud de Extracción.
Revisores de la Solicitud de Extracción
La Sección de revisores muestra los usuarios asignados para revisar esta Solicitud de Extracción. El estado de cada revisor se indica con un ícono que muestra (revisión pendiente, cambios solicitados o aprobado). Consulta las instrucciones a continuación para más detalles sobre el proceso de revisión.
Revisar Solicitud de Extracción
Esperar la verificación de conflictos de fusión
Cuando se crea una Solicitud de Extracción, inicialmente se verifica si hay conflictos de fusión, indicado por 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 en este artículo.
Si la rama tiene conflictos, debes resolver los conflictos 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 asegura la calidad del código, ayuda a identificar conflictos temprano y refuerza los procesos de aprobación del equipo. Todos los revisores deben hacer clic en Aprobar antes de que la Solicitud de Extracción pueda ser fusionada.
Auto-revisión
Si no se agregan revisores, puedes revisar los cambios de tu propia Solicitud de Extracción. No se requiere una revisión o aprobación formal. Después de revisar, puedes fusionar sin aprobación adicional.
En la Sección de revisores sección, haz clic en el ícono 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 se listarán en la Sección de revisores sección, y el estado de la Solicitud de Extracción cambiará a Revisión solicitada.
Revisión de cambios
Para revisar la Solicitud de Extracción, cada revisor debe iniciar sesión en Cloud DevOps y seguir estos pasos:
Las solicitudes de extracción que esperan revisión están marcadas con el estado Revisión solicitada, y los Sección de revisores íconos se muestran.
Haz clic en Solicitudes de extracción desde el menú de la barra lateral izquierda.
Haz clic en la Solicitud de Extracción para revisar.
Haz clic en la pestaña Componentes cambiados .
Haz clic en un componente para revisar.
Todos los componentes deben ser revisados.
Al ver un componente, el código fuente para Destino y Origen se muestra lado a lado con las diferencias resaltadas.
Rojo: Líneas que serán eliminadas.
Verde: Líneas que serán agregadas.
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 origen con los cambios necesarios. Dado que la solicitud no especifica qué debe cambiarse, comunica los cambios específicos requeridos directamente a tu equipo.
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 revisión por el revisor que los solicitó.
Haz clic en la Solicitud de Extracción.
Haz clic en la Actualizar ícono junto al revisor que solicitó la revisión.
Aprobar
Una vez que un revisor haya completado su revisión de todos los componentes y cualquier solicitud de cambio completada, debe aprobar la Solicitud de Extracción antes de que pueda fusionarse.
La Solicitud de Extracción no será aprobada hasta que todos los revisores hagan clic en Aprobar.
Auto-revisión
Al auto-revisarte, no puedes Aprobar la Solicitud de Extracción. Pero dado que no hay revisores, no se requiere aprobación 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 podrá fusionarse.
Siguiente paso: Fusionar
Después de que la Solicitud de Extracción sea aprobada, los cambios de la rama origen están listos para fusionarse en la rama destino. Procede al artículo siguiente para obtener instrucciones sobre el proceso de fusión.
Última actualización
¿Te fue útil?