About pull request reviews

Reviews allow collaborators to comment on the changes proposed in pull requests, approve the changes, or request further changes before the pull request is merged. Repository administrators can require that all pull requests are approved before being merged.

About pull request reviews

After a pull request is opened, anyone with read access can review and comment on the changes it proposes. You can also suggest specific changes to lines of code, which the author can apply directly from the pull request. For more information, see "Reviewing proposed changes in a pull request."

Repository owners and collaborators can request a pull request review from a specific person. Organization members can also request a pull request review from a team with read access to the repository. 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 settings for your team."

Reviews allow for discussion of proposed changes and help ensure that the changes meet the repository's contributing guidelines and other quality standards. You can define which individuals or teams own certain types or areas of code in a CODEOWNERS file. When a pull request modifies code that has a defined owner, that individual or team will automatically be requested as a reviewer. For more information, see "About code owners."

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

Header of review requesting changes with line comments

A review has three possible statuses:

  • Comment: Submit general feedback without explicitly approving the changes or requesting additional changes.
  • Approve: Submit feedback and approve merging the changes proposed in the pull request.
  • Request changes: Submit feedback that must be addressed before the pull request can be merged.

Image of review statuses

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.

You can view all of the reviews a pull request has received in the Conversation timeline, and you can see reviews by repository owners and collaborators in the pull request's merge box.

Image of reviews in a merge box

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

Descubrir y navegar por los debates

Puedes descubrir un navegar por todos los debates dentro de tu solicitud de cambios utilizando el menú de Conversaciones que se muestra en la parte superior de la pestaña Archivos que cambiaron.

Desde esta vista puedes ver qué debates están aún sin resolver, resueltos y desactualizados. Esto te facilita descubrirlos y resolverlos.

Mostrar el menú de debates

Re-requesting a review

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 .

Required reviews

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. For more information, see "About protected branches."

Tip: If necessary, people with admin or write access to a repository can dismiss a pull request review. For more information, see "Dismissing a pull request review."

Further reading

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