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.
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".
Starting a review
-
En el nombre del repositorio, haga clic en Solicitudes de incorporación de cambios.
-
En la lista de solicitudes de incorporación de cambios, haga clic en la que quiera revisar.
-
On the pull request, click Files changed.
-
Optionally, filter the files to show only the files you want to review. For more information, see "Filtering files in a pull request."
-
Mantén el puntero sobre la línea de código donde quieres agregar un comentario y haz clic en el icono de comentario azul. Para agregar un comentario en varias líneas, haz clic y arrastra para seleccionar el intervalo de líneas y, luego, haz clic en el icono de comentario azul.
-
Teclea tu comentario en el campo de comentario.
-
Optionally, to suggest a specific change to the line or lines, click , then edit the text within the suggestion block.
-
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.
Reviewing dependency changes
Nota: La revisión de dependencias se encuentra actualmente en versión beta y está sujeta a cambios.
If the pull request contains changes to dependencies you can use the dependency review for a manifest or lock file to see what has changed and check whether the changes introduce security vulnerabilities. For more information, see "Reviewing dependency changes in a pull request."
-
On the pull request, click Files changed.
-
On the right of the header for a manifest or lock file, display the dependency review by clicking the rich diff button.
-
Puede que también quieras revisar el diff origen, ya que podría haber cambios en el archivo de bloqueo o de manifiesto que no cambian de dependencia o podrían haber dependencias que GitHub no puede procesar, las cuales, como resultado, no aparecen en la revisión de dependencias.
Para regresar a la vista de diferencias de origen, haga clic en el botón .
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.
-
On the pull request, click Files changed.
-
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.
-
On the pull request, click Files changed.
-
Encima del código cambiado, haga 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.
-
Haga clic en Enviar 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.