Pull Request Etapa 2: Revisão

Visão geral

Este artigo explica como completar o processo de Revisão para um Pull Request, que envolve tanto o criador quanto os revisores designados. Conduzir revisões cuidadosas é vital para manter a qualidade do código.

Melhores práticas

• Sempre adicione pelo menos um revisor para garantir responsabilidade.

• Use o campo de descrição para resumir as principais alterações para os revisores.

• Faça merge somente quando todos os conflitos estiverem resolvidos e os testes tiverem passado.

• Use Solicitar alterações em vez de simplesmente não aprovar, pois isso mantém um fluxo de trabalho claro.

• Evite autoavaliação e merge sem um revisor, a menos que seja uma correção crítica com tempo curto.

Guia Visão Geral do Pull Request

Os detalhes de um Pull Request ajudam revisores e contribuintes a obter contexto sobre a solicitação. A Visão geral aba tem duas seções para revisar: Detalhes do Pull Request e Revisores do Pull Request.

Detalhes do Pull Request

Campo	Descriçã
Status	Estado atual do Pull Request (Criado, Revisão solicitada, Alterações solicitadas, e Aprovado)
Branch de destino	A branch onde as alterações serão mescladas.
Branch de origem	A branch onde as alterações se originam.
Criado por	Usuário que criou o Pull Request.
Data de criação	Quando o Pull Request foi criado pela primeira vez.
Data de atualização	Quando ocorreu a última atividade ou alteração no Pull Request.
Descrição	Resumo das alterações deste Pull Request. Escrito pelo criador do Pull Request.

Revisores do Pull Request

O Revisores a seção mostra os usuários designados para revisar este Pull Request. O status de cada revisor é indicado por um ícone que ilustra (revisão pendente, alterações solicitadas ou aprovado). Veja as instruções abaixo para mais detalhes sobre o processo de revisão.

Revisar Pull Request

1

Aguardar verificação de conflitos de merge

1. Quando um Pull Request é criado, ele inicialmente verifica conflitos de merge, indicado por um banner cinza.

   
2. Aguarde o banner mudar para um dos seguintes:

   1. Verde: Esta branch pode ser mesclada

   2. Laranja: Esta branch tem conflitos que devem ser resolvidos

3. Se a branch puder ser mesclada, você pode continuar a revisar o Pull Request seguindo as instruções neste artigo.

   
4. Se a branch tiver conflitos, você deve resolver os conflitos antes de continuar a revisar este Pull Request. Veja o artigo a seguir para aprender como resolver conflitos.

   1. Resolvendo Conflitos de Pull Request

2

(Opcional) Adicionar revisores

Adicionar revisores garante a qualidade do código, ajuda a identificar conflitos cedo e reforça os processos de aprovação da equipe. Todos os revisores devem clicar em Aprovar antes que o Pull Request possa ser mesclado.

Autoavaliação

Se nenhum revisor for adicionado, você pode revisar suas próprias alterações do Pull Request. Nenhuma revisão formal ou aprovação é necessária. Após revisar, você pode fazer o merge sem aprovação adicional.

1. Na Revisores seção, clique no Lápis ícone.

   
2. Clique em Adicionar para cada usuário que você deseja que revise este Pull Request.

   
3. Clique em Salvar.

   
4. Os revisores adicionados serão listados na Revisores seção, e o status do Pull Request mudará para Revisão solicitada.

   

3

Revisar alterações

Para revisar o Pull Request, cada revisor deve entrar no Cloud DevOps e seguir estes passos:

Pull requests aguardando revisão são marcados com o status Revisão solicitada, e os Revisores' ícones são exibidos.

Autoavaliação

Se você estiver fazendo autoavaliação, ainda deve revisar cada componente antes de mesclar.

1. Clique em Pull requests no menu da barra lateral esquerda.

2. Clique no Pull Request para revisar.

   
3. Clique na Guia Componentes alterados .

4. Clique em um componente para revisar.

   

Todos os componentes devem ser revisados.

1. Ao visualizar um componente, o código-fonte para Destino e Origem é exibido lado a lado com as diferenças destacadas.

   1. Vermelho: Linhas a serem removidas.

   2. Verde: Linhas a serem adicionadas.

   
2. Repita o processo de revisão para cada componente.

4

(Opcional) Solicitar alterações

Ao revisar, use Solicitar alterações para indicar que correções são necessárias antes da aprovação. Esse status solicita que o criador do Pull Request atualize a branch de origem com as mudanças necessárias. Como a solicitação não especifica o que precisa ser alterado, comunique as mudanças específicas necessárias diretamente à sua equipe.

Autoavaliação

Se você estiver fazendo autoavaliação, você não pode Solicitar alterações.

1. Clique em Solicitar alterações.

   
2. Após solicitar alterações, o status do Pull Request mudará para Alterações solicitadas.

   

Reenviar para revisão

Após fazer as alterações solicitadas, reenvie o Pull Request para revisão pelo revisor que as solicitou.

1. Clique no Pull Request.

   
2. Clique na Atualizar ícone ao lado do revisor que solicitou a revisão.

   

5

Aprovar

Uma vez que um revisor tenha concluído sua revisão de todos os componentes e quaisquer solicitações de alteração concluídas, ele deve aprovar o Pull Request antes que ele possa ser mesclado.

O Pull Request não será aprovado até que todos os revisores cliquem em Aprovar.

Autoavaliação

Ao fazer autoavaliação, você não pode Aprovar o Pull Request. Mas como não há revisores, a aprovação não é necessária antes do merge.

1. Clique em Aprovar.

   
2. Depois que todos os revisores aprovarem, o status do Pull Request mudará para Aprovado, e ele poderá ser mesclado.

   

6

Próximo passo: Mesclagem

Após a aprovação do Pull Request, as alterações da branch de origem estarão prontas para serem mescladas na branch de destino. Prossiga para o artigo a seguir para obter instruções sobre o processo de merge.

• Pull Request Etapa 3: Mesclagem