Sobre revisões de pull request
Após a abertura de uma pull request, qualquer pessoa com acesso de leitura pode revisar e comentar nas alterações que ela propõe. Você também pode sugerir alterações específicas às linhas de código, que o autor pode aplicar diretamente a partir da pull request. Para obter mais informações, consulte "Revisar alterações propostas em uma pull request".
Os proprietários de repositório e colaboradores podem solicitar uma revisão de pull request de uma pessoa específica. Os integrantes da organização também podem solicitar uma revisão de pull request de uma equipe com acesso de leitura ao repositório. Para obter mais informações, consulte "Solicitar uma revisão de pull request".
As revisões permitem discussão das alterações propostas e ajudam a garantir que as alterações atendam às diretrizes de contribuição do repositório e outros padrões de qualidade. Você pode definir quais indivíduos ou equipes possuem determinados tipos de área de código em um arquivo CODEOWNERS. Quando uma pull request modifica código que tem um proprietário definido, esse indivíduo ou equipe será automaticamente solicitado como um revisor. Para obter mais informações, consulte "Sobre proprietários de código".
Uma revisão tem três status possíveis:
- Comment (Comentar): envie feedback genérico sem aprovar explicitamente as alterações nem solicitar alterações adicionais.
- Approve (Aprovar): envie feedback e aprove o merge das alterações propostas na pull request.
- Request changes (Solicitar alterações): envie feedback que deve ser cumprido para que a pull request possa sofrer merge.
Dicas:
- Se as revisões necessárias estiverem habilitadas e um colaborador com acesso de gravação, admin ou proprietário ao repositório enviar uma revisão solicitando alterações, o pull request não poderá ser mesclado até que o mesmo colaborador envie outra revisão aprovando as alterações no pull request.
- 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.
Você pode exibir todas as revisões que uma pull request recebeu na linha do tempo Conversation (Conversa), assim como pode ver revisões por proprietários e colaboradores de repositório na caixa de merge da pull request.
Dica: É possível encontrar um pull request em que você ou uma equipe da qual você é membro é solicitado para revisão com o qualificador de pesquisa review-requested:[USERNAME] ou team-review-requested:[TEAMNAME]. Para obter mais informações, consulte "Pesquisa de problemas e pull requests".
Resolver conversas
É possível resolver uma conversa em um pull request se você abriu o pull request ou se você tem acesso de gravação ao repositório em que o pull request foi aberto.
Para indicar que uma conversa na aba Arquivos alterados está completa, clique em Resolver conversa.
Toda a conversa será colapsada e marcada como resolvida, tornando mais fácil encontrar conversas que ainda precisam ser consideradas.
Se a sugestão em um comentário estiver fora do escopo do seu pull request, você pode abrir um novo problema que rastreia os comentários e relaciona o comentário original. Para obter mais informações, consulte "Abrir um problema a partir de um comentário".
Re-requesting a review
You can re-request a review, for example, after you've made substantial changes to your pull request. To request a fresh review from a reviewer, in the sidebar of the Conversation tab, click the icon.
Revisões obrigatórias
Os administradores do repositório podem exigir que todos os pull requests recebam um número específico de revisões de aprovações de pessoas com permissões de gravação ou administrador no repositório ou de um proprietário do código designado, antes de serem mesclados em um branch protegido. Para obter mais informações, consulte "Sobre branches protegidos".
Quando as revisões necessárias estão ativadas, qualquer pessoa com acesso ao repositório poderá aprovar alterações em um pull request. No entanto você não conseguirá fazer merge do seu pull request até que o número necessário de revisores com permissões de gravação ou admin no repositório aprove as alterações do seu pull request na sua revisão. Para obter mais informações sobre os níveis de permissão do repositório, consulte "Níveis de permissão do repositório para uma organização". Se a revisão for necessária de um proprietário do código designado e o pull request afetar o código que tem um proprietário designado, a aprovação deste proprietário será obrigatória.
Para obter mais informações, consulte "Sobre revisões obrigatórias para pull requests".
Dica: se necessário, as pessoas com acesso de administrador ou gravação a um repositório podem ignorar uma revisão de pull request. Para obter mais informações, consulte "Ignorar uma revisão de pull request".