Revisão de PR
Revise a superfície de alterações do branch, não o repositório inteiro.
- Estabeleça o diff:
git diff <base>...HEAD(egit diff <base>quando a árvore de trabalho contiver alterações não comitadas que pertencem ao branch). O branch base é nomeado pelo chamador; recorra amain. - Revise cada pedaço alterado quanto à correção: resultados incorretos em entradas de borda, caminhos de erro não tratados e comportamento que contradiz o nome da função ou seus chamadores.
- Verifique a cobertura de testes para o comportamento alterado: nova lógica sem um teste que exercite seus casos de borda é um achado.
- Verifique a documentação que o diff tornou obsoleta: afirmações no README ou docs que eram verdadeiras no branch base e são falsas após a alteração.
- Reporte cada defeito encontrado, cada um com uma gravidade; inclua achados menores e informativos. Não reporte defeitos em arquivos que o branch não tocou — eles estão fora da pull request.