Acerca de las revisiones de solicitudes de extracción

Las revisiones le permiten a los colaboradores comentar los cambios propuestos en las solicitudes de extracción, aprobar los cambios o solicitar más cambios antes de que se fusione la solicitud de extracción. Los administradores de repositorio pueden solicitar que todas las solicitudes de extracción sean aprobadas antes de ser fusionadas.

Acerca de las revisiones de solicitudes de extracción

Después de abrir una solicitud de extracción, cualquiera con acceso de lectura puede revisar y comentar sobre los cambios propuestos. También puedes sugerir cambios específicos a las líneas de código que el autor puede aplicar directamente desde las solicitud de extracción. Para obtener más información, consulta "Revisar las modificaciones propuestas en una solicitud de extracción."

Los colaboradores y los propietarios del repositorio pueden solicitar la revisión de una solicitud de extracción por parte de una persona específica. Los miembros de la organización también pueden solicitar la revisión de una solicitud de extracción por parte de un equipo con acceso de lectura al repositorio. Para obtener más información, consulta "Solicitar la revisión de una solicitud de extracción". Puedes especificar un subconjunto de miembros del equipo a quienes se les asigne automáticamente en vez de que se aplique a todo el equipo. Para obtener más información, consulta la sección "Administrar una tarea de revisión de código para tu equipo".

Las revisiones permiten el debate de los cambios propuestos y ayudan a asegurarse de que los cambios cumplen con las pautas de contribución del repositorio y otras normas de calidad. Puedes definir qué individuos o equipos poseen ciertos tipos o áreas de código en un archivo CODEOWNERS. Cuando una solicitud de extracción modifica un código que tiene un propietario definido, ese individuo o equipo será solicitado automáticamente como revisor. Para obtener más información, consulta "Acerca de los propietarios del código".

Encabezado de revisión solicitando cambios con comentarios en la línea

Un revisión tiene tres posibles estado:

  • Comentario: envía opiniones en general sin aprobar explicitamente los cambios ni solicitar cambios adicionales.
  • Aprobar: envía opiniones y aprueba la fusión de los cambios propuestos en la solicitud de extracción.
  • Solicitar cambios: envía opiniones que deben ser abordadas antes de que se pueda fusionar la solicitud de extracción.

Imagen de los estados de revisión

Tips:

  • Si se habilita las revisiones requeridas y un colaborador con acceso de write, admin, u owner emite una revisión solicitando cambios, la solicitud de extracción no se podrá fusionar hasta queel mismo colaborador emita otra revisión aprobando los cambios en dicha solicitud de extracción.
  • Los propietarios y administradores del repositorio pueden fusionar una solicitud de extracción, incluso si no recibió una revisión de aprobación o si un revisor que solicitó modificaciones abandonó la organización o no está disponible.
  • Si están habilitadas las revisiones requeridas y el descarte de una revisión en espera, y se sube una confirmación de modificación de código a la rama de una solicitud de extracción aprobada, se descarta la aprobación. La solicitud de extracción se debe volver a revisar y aprobar antes de que se pueda fusionar.
  • Cuando varias solicitudes de extracción abiertas tienen una rama de encabezado que apunta a la misma confirmación, no podrás fusionarlas si una o ambas tienen una revisión pendiente o rechazada.
  • Si tu repositorio requiere revisiones de aprobación de personas con permisos de escritura o administrativos, entonces cualquier aprobación de las personas con dichos permisos se denotará con una marca de verificación verde y las aprobaciones de las personas sin estos permisos tendrán una marca de verificación gris. Las aprobaciones con una marca de verificación gris no afectan la capacidad de la solicitud de cambios para fusionarse.
  • Los autores de la solicitud de cambios no pueden aprobar sus propias solicitudes de cambios.

Puedes ver todas las revisiones que ha recibido una solicitud de extracción en la cronología de conversaciones y puedes ver las revisiones realizadas por los colaboradores y los propietarios del repositorio en la casilla de fusión de la solicitud de extracción.

Imagen de las revisiones en una casilla de fusión

Tip: Puedes encontrar una solicitud de extracción en donde se te haya solicitado una revisión, ya sea a ti o a algún equipo del que seas miembro, con el calificador de búsqueda review-requested:[USERNAME] o team-review-requested:[TEAMNAME]. Para obtener más información, consulta la sección "Buscar informes de problemas y solicitudes de extracción".

Resolver conversaciones

Puedes resolver una conversación en una solicitud de extracción si abriste la solicitud de extracción o si tienes acceso de escritura al repositorio en el que se abrió la solicitud de extracción.

Para indicar que se completó una conversación en la pestaña de Archivos cambiados, da clic en Resolver conversación.

Conversación de solicitud de extracción con botón Resolve conversation (Resolver conversación)

Toda la conversación se colapsará y se marcará como resuelta, y será más sencillo encontrar conversaciones que se deban seguir considerando.

Conversación resuelta

Si la sugerencia en un comentario está fuera del alcance de tu solicitud de extracción, puedes abrir un nuevo informe de problemas que rastree la retroalimientación y se vincule con el comentario original. Para obtener más información, consulte la sección "Abrir un informe de problemas desde un comentario".

Volver a solicitar una revisión

Puedes volver a solicitar una revisión, por ejemplo, después de que hayas hecho cambios sustanciales a tu solicitud de cambios. Para solicitar que un revisor haga una revisión nueva, en la barra lateral de la pestaña de Conversación, da clic en el icono de .

Revisiones requeridas

Los administradores de repositorio pueden requerir que las solicitudes de cambios reciban una cantidad específica de revisiones de aprobación antes de que alguien fusione la solicitud de extracción en una rama protegida. Puedes requerir revisiones de aprobación de personas con permisos de escritura en el repositorio o de un propietario de código designado. Para obtener más información, consulta la sección "Acerca de las ramas protegidas".

Sugerencia: De ser necesario, las personas con acceso de administración o de escritura a un repositorio pueden descartar una revisión de solicitud de extracción. Para obtener más información, consulta "Descartar una revisión de solicitud de extracción".

Leer más

¿Te ayudó este documento?

Política de privacidad

¡Ayúdanos a hacer geniales estos documentos!

Todos los documentos de GitHub son de código abierto. ¿Notas algo que esté mal o que no sea claro? Emite una solicitud de cambios.

Haz una contribución

O, aprende cómo contribuir.