Obter comentários com solicitações de pull

As solicitações pull dão suporte à revisão e à mesclagem de código em um único processo colaborativo. Depois que um desenvolvedor adiciona um recurso ou uma correção de bug, ele cria uma solicitação de pull para iniciar o processo de mesclagem das alterações no branch upstream. Em seguida, outros membros da equipe têm a chance de revisar e aprovar o código antes que ele seja finalizado. Use solicitações de pull para examinar os trabalhos em andamento e obter comentários antecipados em relação às alterações. Mas não há compromisso de mesclar as mudanças. Um proprietário pode abandonar um pull request a qualquer momento.

Obter código revisado

A revisão de código feita como parte de uma solicitação de pull não é apenas para localizar bugs óbvios; é para isso que são os testes. Uma boa revisão de código captura problemas menos óbvios que podem levar a problemas caros mais tarde.

As revisões de código ajudam a proteger a equipe contra mesclagens incorretas e builds quebrados que reduzem a produtividade da equipe. Revisões capturam problemas antes da mesclagem, protegendo branches importantes contra alterações indesejadas.

As revisões de código também incentivam e fortalecem a colaboração e a comunicação entre os desenvolvedores. E a equipe obtém um registro claro de todas as alterações feitas entre o branch principal e os branches de funcionalidades.

Facilite a troca de expertise e compartilhe estratégias de resolução de problemas utilizando uma ampla variedade de revisores em revisões de código. Difundir habilidades e conhecimentos torna a equipe mais forte e resiliente.

Dê excelentes comentários

As revisões de alta qualidade começam com comentários de alta qualidade. As chaves para ótimos comentários em um pull request são:

• Garanta que as pessoas certas revisem o pull request.
• Verifique se os revisores sabem o que o código faz.
• Forneça comentários construtivos e acionáveis.
• Responda aos comentários em tempo hábil.

Ao atribuir revisores a uma solicitação de pull, selecione o conjunto certo de revisores. Os revisores devem saber como o código funciona, mas também incluir desenvolvedores que trabalham em outras áreas para que possam compartilhar suas ideias.

Forneça uma descrição clara das alterações e forneça um build do código que tenha a correção ou o recurso funcionando nele. Os revisores devem fazer um esforço para fornecer comentários sobre as alterações com as quais não concordam. Identifique o problema e dê sugestões específicas sobre o que poderia ser feito de forma diferente. Esse feedback tem uma intenção clara e é fácil para o proprietário do pedido de pull request entender.

O proprietário da solicitação de pull deve responder a comentários, aceitar sugestões ou explicar por que ele se recusa a aplicá-las. Algumas sugestões são boas, mas podem estar fora do escopo do pull request. Pegue essas sugestões e crie novos itens de trabalho e branches de funcionalidades separados do pull request para realizar essas alterações.

Proteger branches com políticas

Há alguns branches críticos em um repositório nos quais as equipes precisam que estejam sempre em bom estado, como o branch main. O Teams pode exigir solicitações de pull para fazer alterações nessas ramificações com plataformas como o GitHub e o Azure DevOps. Os desenvolvedores que enviarem alterações diretamente para as ramificações protegidas terão seus pushes rejeitados.

Adicione condições adicionais aos pull requests para garantir um nível mais elevado de qualidade de código em branches principais. A aprovação de vários revisores e uma compilação limpa do código mesclado são alguns requisitos extras geralmente empregados para proteger branches principais.

Saiba Mais

O GitHub tem uma extensa documentação sobre como propor alterações em seu trabalho com solicitações de pull.

Leia mais sobre como fazer ótimos comentários em revisões de código e usar modelos de solicitação de pull para fornecer diretrizes aos revisores. O Azure DevOps também oferece uma experiência de solicitação de pull avançada que é fácil de usar e dimensiona conforme necessário.