Reviewing proposed changes in a pull request

In a pull request, you can review and discuss commits, changed files, and the differences (or "diff") between the files in the base and compare branches.

About reviewing pull requests

You can review changes in a pull request one file at a time. While reviewing the files in a pull request, you can leave individual comments on specific changes. After you finish reviewing each file, you can mark the file as viewed. This collapses the file, helping you identify the files you still need to review. A progress bar in the pull request header shows the number of files you've viewed. After reviewing as many files as you want, you can approve the pull request or request additional changes by submitting your review with a summary comment.

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

Starting a review

  1. Debajo del nombre de tu repositorio, da clic en Solicitudes de extracción. Selección de la pestaña de propuestas y solicitudes de extracción
  2. En la lista de solicitudes de extracción, haz clic en la solicitud de extracción que deseas revisar.
  3. En la solicitud de extracción, da clic en Archivos cambiados. Pestaña de Archivos Cambiados
  4. Pasa el puntero sobre la línea de código en la que deseas agregar un comentario y da clic en el icono de comentario azul. Para agregar un comentario en líneas múltiples, da clic y arrastra para seleccionar el rango de líneas, luego da clic en el icono de comentario azul. Icono de comentario azul
  5. Teclea tu comentario en el campo de comentario. Campo de comentario
  6. Opcionalmente, para sugerir un cambio específico a la línea o líneas,da clic en, luego edita el texto dentro del bloque de sugerencia. Bloque de sugerencia
  7. When you're done, click Start a review. If you have already started a review, you can click Add review comment. Start a review button

Before you submit your review, your line comments are pending and only visible to you. You can edit pending comments anytime before you submit your review. To cancel a pending review, including all of its pending comments, scroll down to the end of the timeline on the Conversation tab, then click Cancel review.

Cancel review button

Marking a file as viewed

After you finish reviewing a file, you can mark the file as viewed, and the file will collapse. If the file changes after you view the file, it will be unmarked as viewed.

  1. En la solicitud de extracción, da clic en Archivos cambiados. Pestaña de Archivos Cambiados
  2. On the right of the header of the file you've finished reviewing, select Viewed. Viewed checkbox

Submitting your review

After you've finished reviewing all the files you want in the pull request, submit your review.

  1. En la solicitud de extracción, da clic en Archivos cambiados. Pestaña de Archivos Cambiados
  2. Sobre el código que ha cambiado, da clic en Revisar cambios. Botón para revisar cambios
  3. Teclea un comentario que resuma tu retroalimentación sobre los cambios propuestos. Ventana de comentarios para el resumen de la revisión
  4. Select the type of review you'd like to leave: Radio buttons with review options
    • Select Comment to leave general feedback without explicitly approving the changes or requesting additional changes.
    • Select Approve to submit your feedback and approve merging the changes proposed in the pull request.
    • Select Request changes to submit feedback that must be addressed before the pull request can be merged.
  5. Da clic en Emitir 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.

Further reading

Did this doc help you?Privacy policy

Help us make these docs great!

All GitHub docs are open source. See something that's wrong or unclear? Submit a pull request.

Make a contribution

O, learn how to contribute.