Por que precisamos ter revisão de código ?
- Pegar bugs, quanto mais cedo um bug é encontrado, mais barato é o custo de remoção.
Pegar problemas com a qualidade do código, a medida que o código aumenta, é natural que a qualidade de código vá diminuir, para isso, a revisão pode dificultar que isso aconteça. O custo de desenvolvimento fica mais barato fazendo do jeito certo na primeira, em vez de prometer refatorar o código futuramente.
Oportunidade de aprendizado para o criador e os revisores do Pull Request. Quando fomentamos a discussão sobre as mudanças no código, estamos pegando o feedback de cada um dos envolvidos para refinar o pull request, assim tanto quem é o autor, como os revisores podem aprender um com o outro.
Uma chance para o time se alinhar, os desenvolvedores são forçados a se comunicarem e entender mais sobre o contexto do novo código, assim dá oportunidade para eles se alinharem e tomarem mais contextos sobre os novos comportamentos do software.
Escrever software com uma melhor qualidade.
Revisões de código são potencializadores, para o bem e para o mal.
Se a empresa tiver uma cultura organizacional saudável, ele poderá trazer alguns ou todos os benefícios citados anterior, ampliando o bem.
No entanto se a cultura da empresa é ruim, ou integrantes com personalidades difíceis, ou comportamentos não profissionais, isso será potencializado durante as revisões. Nesse caso a revisão não compensa ser feita, pois entrarão em discussões desnecessárias que retardam o desenvolvimento, confrontos, comentários arrogantes ou condescendentes, que levará a diminuição da moral, o medo gerado por tais comentários negativos poderá levar os desenvolvedores a esconder defeitos, e no pior cenário, irá contribuir para a rotatividade da equipe, pois os integrantes deixarão a empresa por frustrações.
O que precisamos ter para participar de uma revisão de código
Top comments (2)
Perfeitas as considerações. Uma coisa que ajuda a focar no que é importante durante a revisão de código é o projeto ter um linter configurado no CI, assim o revisor não precisa ficar verificando e comentando se tá com aspa duplas ou simples, se tá com a indentação correta, se colocou uma virgula ou parênteses e etc. o foco fica na solução do código.
Ola Stephann, publiquei um artigo justamente apontando isso que voce comentou, la eu falo sobre o sonarlint, ele me ajudou muito a melhorar um pouco o codigo antes do commit.
dev.to/paulowalravendev/estabelece...