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.
Dica: encontre uma solicitação de pull na qual você ou uma equipe da qual você é membro é solicitada para revisão com o qualificador de pesquisa review-requested:[USERNAME]
ou team-review-requested:[TEAMNAME]
. Para obter mais informações, confira "Pesquisar problemas e solicitações de pull".
Starting a review
-
Abaixo do nome do repositório, clique em Solicitações de pull.
-
Na lista de solicitações pull, clique na solicitação pull que você quer 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."
-
Passe o mouse sobre a linha de código em que você gostaria de adicionar um comentário e clique no ícone de comentário azul. Para adicionar um comentário em várias linhas, clique e arraste para selecionar o intervalo de linhas e clique no ícone de comentário azul.
-
No campo comentário, digite o seu comentário.
-
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
Observação: atualmente, a revisão de dependência está em versão beta e sujeita a alterações.
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.
-
É possível que você também queira revisar o diff da fonte, porque pode haver alterações no manifesto ou arquivo de bloqueio que não mudam as dependências, ou pode haver dependências que GitHub não pode analisar e que, consequentemente, não aparecem na revisão de dependências.
Para retornar ao modo de exibição de diferenciação de origem, clique no botão .
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.
-
Acima do código alterado, clique em Revisar alterações.
-
Digite um comentário que resume o seu feedback sobre as alterações propostas.
-
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.
-
Clique em Enviar revisão.
Dicas:
- Se as revisões necessárias estiverem habilitadas e um colaborador com acesso de gravação, administrador ou proprietário no repositório enviar uma revisão solicitando alterações, a solicitação de pull não poderá ser mesclada até que o mesmo colaborador envie outra revisão aprovando as alterações na solicitação de pull.
- Proprietários e administradores do repositório podem fazer merge de um pull request mesmo que não tenham recebido uma revisão de aprovação; ou se um revisor que solicitou alterações saiu da organização ou estiver indisponível.
- Se as revisões necessárias e o descarte de uma revisão obsoleta estiverem habilitados e um commit de modificação de código for enviado para o branch de um pull request aprovado, a aprovação será ignorada. O pull request deve ser revisado e aprovado novamente antes de poder ser mesclado.
- Quando vários pull requests abertos têm um branch principal que aponta para o mesmo commit, você não conseguirá mesclá-los se um ou ambos tiverem uma revisão pendente ou rejeitada.
- Se o repositório exigir a aprovação de revisões de pessoas com permissões de gravação ou de administrador, todas as aprovações de pessoas com essas permissões serão indicadas com uma marca de seleção verde e as aprovações de pessoas sem essas permissões têm uma marca de seleção cinza. As aprovações com uma marca de seleção cinza não afetam a possibilidade de mesclagem da solicitação de pull.
- Os autores de pull request não podem aprovar seus próprios pull requests.