Processo de Code Review

Neste capítulo, veremos todos os passos para que o Code Review seja realizado da melhor forma, garantindo e mantendo a qualidade de nossos códigos e, consequentemente, do sistema.

O que é Code Review?

Code Review, ou CR, é a ato de revisar o código da equipe. Esta parte do processo é de extrema importância para a qualidade e a estabilidade do sistema, pois é nela que os engenheiros conseguem analisar os códigos de seu time, e prevenir possíveis erros e desvio de padrões de códigos definidos.

Quem faz Code Review?

Em suma, os seniores do time ficam responsáveis pelos Code Reviews. Isso se dá pelo fato de que, quem possui mais experiência, consegue analisar e apontar melhor as boas práticas utilizadas, bem como a falta das mesmas. Entretanto, nada impede que um pleno faça uma CR de um júnior, pois a "hierarquia" de conhecimento se mantém.

Além do Code Review

Junto ao CR, não obrigatório "by the book", mas muito importante, é o processo de testes da PR.

Além de analisarmos o código dos colegas, sempre é bom testarmos a PR como um todo, garantindo assim o bom funcionamento da feature/fix, evitando que a esteira seja quebrada lá na frente.

Geralmente, este passo é chamado de "teste integrado", pois o código é realmente colocado à prova, com todas as integrações que ela possua.

Passos do Code Review

Um bom Code Review segue alguns passos, tanto para quem cria a PR, quanto para quem a revisa de fato. Abaixo, vamos ver todos os passos para um CR consistente:

• Criação da PR (Pull Request);

• Visibilidade da PR ao time;

• Realização do CR/Teste integrado;

• Aprovação da PR;

• Reprovação da PR.