Skip to main content

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 incorporación de cambios, cualquiera con acceso de lectura puede revisar y comentar 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 más información, vea "Revisión de los cambios propuestos en una solicitud de incorporación de cambios".

Predeterminadamente, en los repositorios públicos, cualquier usuario puede emitir revisiones que aprueben o soliciten cambios a una solicitud de cambios. Los propietarios de la organización y los administradores del repositorio pueden limitar quién puede dar revisiones de solicitudes de incorporación de cambios aprobadas o cambios de solicitud. Para más información, vea "Administración de revisiones de solicitudes de incorporación de cambios en la organización" y "Administración de revisiones de solicitudes de incorporación de cambios en el repositorio".

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 más información, vea "Solicitud de la revisión de una solicitud de incorporación de cambios". Puedes especificar un subconjunto de miembros del equipo que se asignarán automáticamente en lugar de todo el equipo. Para más información, consulta "Administración de la configuración de revisión del código para el 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 más información, vea "Acerca de los propietarios del código".

Puede programar recordatorios para las solicitudes de incorporación de cambios que se deban revisar. Para más información, vea "Administración de recordatorios programados para solicitudes de incorporación de cambios".

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

Un revisión tiene tres posibles estado:

  • Comentario: se envían comentarios generales sin aprobar de manera explícita los cambios ni solicitar cambios adicionales.
  • Aprobar: se envían comentarios y se aprueba la combinación de los cambios propuestos en la solicitud de incorporación de cambios.
  • Solicitar cambios: se envían comentarios que se deben abordar antes de que se pueda combinar la solicitud de incorporación de cambios.

Imagen de los estados de revisión

Sugerencias:

  • Si las revisiones obligatorias están habilitadas y un colaborador con acceso de escritura, administración o administración al repositorio envía una revisión de solicitud de cambios, la solicitud de incorporación de cambios no se puede combinar hasta que el mismo colaborador envíe otra revisión en la que se aprueben las modificaciones de la solicitud de incorporación de cambios.
  • 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

Propina: Puedes encontrar una solicitud de incorporación de cambios en la que se te solicite a ti o a un equipo del que formes parte una revisión con el calificador de búsqueda review-requested:[USERNAME] o team-review-requested:[TEAMNAME]. Para más información, consulta "Búsqueda de incidencias y solicitudes de incorporación de cambios".

Resolving conversations

You can resolve a conversation in a pull request if you opened the pull request or if you have write access to the repository where the pull request was opened.

To indicate that a conversation on the Files changed tab is complete, click Resolve conversation.

Pull request conversation with Resolve conversation button

The entire conversation will be collapsed and marked as resolved, making it easier to find conversations that still need to be addressed.

Resolved conversation

If the suggestion in a comment is out of your pull request's scope, you can open a new issue that tracks the feedback and links back to the original comment. For more information, see "Opening an issue from a comment."

Discovering and navigating conversations

You can discover and navigate to all the conversations in your pull request using the Conversations menu that's shown at the top of the Files Changed tab.

From this view, you can see which conversations are unresolved, resolved, and outdated. This makes it easy to discover and resolve conversations.

Showing the conversations menu

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 realice una revisión nueva, en la barra lateral de la pestaña Conversation, haga clic en el icono .

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 más información, vea "Acerca de las ramas protegidas".

Sugerencia: Si es necesario, los usuarios con acceso de administrador o escritura a un repositorio pueden descartar una revisión de solicitud de incorporación de cambios. Para más información, vea "Descarte de la revisión de una solicitud de incorporación de cambios".

Información adicional