Executando verificação de segurança...
Em resposta a [DÚVIDA] Pull Requests são realmente necessárias?
4
Pilati

Ao ter que fazer a revisão de outro código cada dev vai acabar entregando menos código e aumentando o tempo das entregas

Qual a qualidade da sua base de código hoje?

Qual a porcentagem de tempo que é gasta em correção de bugs?

Quando desenvolvemos um código temos uma visão enviesada do resultado, testamos um numero X de cenários e não conseguimos pensar em cenários diferentes. Quantas vezes você apenas explicou um problema para outra pessoa e esta te deu um cenário que você nunca imaginou?

A revisão pode aumentar o tempo da entrega, mas a entrega vai ser com uma qualidade maior e consequentemente vai diminuir a quantidade de bugs e o tempo gasto em correções

Disclaimer

Revisão de código e pull request são duas coisas diferentes.

Pull Request é simplesmente o ato de solicitar o merge dos commits ao repositório principal sem a ocorrencia de conflitos de merge.

Revisão é analisar a qualidade dessa solicitação

Carregando publicação patrocinada...
2
giovanniangelo

Qual a qualidade da sua base de código hoje?

Boa pergunta, essa seria o meu próximo post aqui no tabnews rs "Como avaliar a qualidade de um código?"

Sobre esse ponto eu comecei a pesquisar recentemente sobre o Sonarqube, mas hoje eu não sei dizer de forma objetiva qual a qualidade do código. Subjetivamente, eu penso que dá pra melhorar, mas hoje não tenho dados pra dizer onde melhorar e como.

Recentemente começamos a usar o Jira para mapear as tarefas, mas ainda tem pouco tempo pra tirar alguma conclusão de lá, mas percebo que com mais dados vou conseguir fazer essa análise um pouco melhor da relação de bugs por falha de código.

A revisão pode aumentar o tempo da entrega, mas a entrega vai ser com uma qualidade maior e consequentemente vai diminuir a quantidade de bugs e o tempo gasto em correções

Foi o ponto que mais fez sentido pra mim pra justificar ter um processo de revisão.

Revisão de código e pull request são duas coisas diferentes.

Realmente, tratei como se fossem sinônimas mas é possível fazer revisão sem pull request, por exemplo com pair programming talvez. Acabei juntando as duas pelo fato do pul request ser bem útil pra revisão.

1
Pilati
Autor
Autor

Sim, no github tem todo um processo de aprovação e rejeição do PR. Dá pra explicar o motivo que foi recusado e voltar.

Como avaliar a qualidade de um código?

Primeira pergunta é a que fiz antes. Qual a % de tempo que sua equipe fica corrigindo problemas e qual a % do tempo é gasto para coisas novas?

Acredito que essa é a primeira resposta que você precisa ter