Qual a melhor maneira de revisar pull requests?
Atualmente trabalho em uma boa empresa, temos o gitflow seguido a risca.
Mas da minha parte acho que deveria revisar melhor os PRs. Gostaria de ter dicas e sugestões para revisar melhor os PRs
Atualmente trabalho em uma boa empresa, temos o gitflow seguido a risca.
Mas da minha parte acho que deveria revisar melhor os PRs. Gostaria de ter dicas e sugestões para revisar melhor os PRs
Práticas recomendadas dentro do próprio Google: https://google.github.io/eng-practices/review/reviewer/
Dependendo do tamanho do PR, o que costumo fazer é rodar o código e ver se funciona conforme deveria.
Isso vai depender de haver uma explicação do que se pretende fazer com aquele código e também de como testar.
Vai depender muito de como está explicado, tanto em código como em escrita na descrição ou na conversação do PR.
Uma boa prática, é se manter discussões dentro do PR, assim tendo um contexto sobre as alterações no futuro, não apenas deixando as explicações em tasks de sistemas que amanhã podem ser deixados de usar ou então em reuniões.