Frecuentemente publicamos actualizaciones de nuestra documentación. Es posible que la traducción de esta página esté en curso. Para conocer la información más actual, visita la documentación en inglés. Si existe un problema con las traducciones en esta página, por favor infórmanos.

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.

En este artículo

Acerca de las revisiones de solicitudes de extracción

Una vez que se abre una solicitud de extracción, cualquiera con acceso de lectura puede revisar y comentar los cambios que se proponen. También puedes sugerir cambios específicos a una línea, que el autor puede aplicar directamente desde la solicitud de extracción. Para obtener más información, consulta "Revisar los cambios propuestos 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. For more information, see "Requesting a pull request review." You can specify a subset of team members to be automatically assigned in the place of the whole team. For more information, see "Managing code review assignment for your team."

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".

You can schedule reminders for pull requests that need to be reviewed. For more information, see "Managing scheduled reminders for pull requests."

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

Sugerencias:

  • If required reviews are enabled and a collaborator with write, admin, or owner access to the repository submits a review requesting changes, the pull request cannot be merged until the same collaborator submits another review approving the changes in the pull request.
  • 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.

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: You can find a pull request where you or a team you're a member of is requested for review with the search qualifier review-requested:[USERNAME] or team-review-requested:[TEAMNAME]. For more information, see "Searching issues and pull requests."

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.

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

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

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. Para obtener más información, consulta "Incorporar opiniones en tu solicitud de extracción".

Revisiones requeridas

Repository administrators can require that all pull requests receive a specific number of approving reviews from people with write or admin permissions in the repository or from a designated code owner before they're merged into a protected branch. Para obtener más información, consulta"Acerca de las ramas protegidas".

Cuando las revisiones requeridas están habilitadas, cualquier persona con acceso al repositorio puede aprobar cambios en una solicitud de extracción. However, you won't be able to merge your pull request until the required number of reviewers with write or admin permissions in the repository approve your pull request's changes in their review. Para obtener más información acerca de los niveles de permiso del repositorio, consulta "Niveles de permiso del repositorio para una organización". Si se requiere la revisión de un propietario de código designado, y la solicitud de extracción afecta el código que tiene un propietario designado, se requiere la autorización de ese propietario.

Para obtener más información, consulta "Acerca de las revisiones requeridas para las solicitudes de extracción".

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

Pregunta a una persona

¿No puedes encontrar lo que estás buscando?

Contáctanos