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
-
Debajo del nombre de tu repositorio, da clic en Solicitudes de cambios.
-
En la lista de solicitudes de extracción, haz clic en la solicitud de extracción que deseas revisar.
-
En la solicitud de extracción, da clic en Archivos cambiados.
-
Optionally, filter the files to show only the files you want to review. For more information, see "Filtering files in a pull request."
-
Pasa el puntero del mouse sobre la línea de código en donde te gustaría agregar un comentario y haz clic en el icono de comentarios azul. Para agregar un comentario en varias líneas, haz clic y arrastra le mouse para seleccionar el rango de líneas y luego haz clic en el icono de comentarios azul.
-
Teclea tu comentario en el campo de comentario.
-
Opcionalmente, para sugerir un cambio específico a la línea o líneas, haz clic en y luego edita el texto dentro del bloque de sugerencias.
-
When you're done, click Start a review. If you have already started a review, you can click Add review comment.
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.
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.
-
En la solicitud de extracción, da clic en Archivos cambiados.
-
On the right of the header of the file you've finished reviewing, select Viewed.
Submitting your review
After you've finished reviewing all the files you want in the pull request, submit your review.
-
En la solicitud de extracción, da clic en Archivos cambiados.
-
Sobre el código que ha cambiado, da clic en Revisar cambios.
-
Teclea un comentario que resuma tu retroalimentación sobre los cambios propuestos.
-
Select the type of review you'd like to leave:
- 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.
-
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.
- 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.