Respostas para as perguntas mais frequentes sobre code review
(brunch.co.kr/@cleancode)"Eu entendo que code review é bom, mas não há tempo. Há muitas outras coisas para fazer além da revisão."
- Organização da resposta para a pergunta acima, a mais frequente que o autor, que dá palestras sobre code review, recebe
- Vamos fazer com que o autor (quem abre o PR) se esforce para que o code review demande o mínimo de tempo possível
- Assim como em uma reunião diária de stand-up de cerca de 10 minutos, vamos reservar horários fixos para revisão: 30 minutos pela manhã e 30 minutos após o almoço
- Qualidade e produtividade:
Se investirmos no começo, podemos reduzir significativamente os custos que surgem mais tarde e, ao diminuir o custo de mudanças futuras, isso leva ao aumento da produtividade - Outros pontos:
Se faltar tempo, vamos começar ao menos pelas partes críticas, como bugs/incidentes, e ampliar gradualmente.
Vamos reconhecer, dentro da organização, o esforço dedicado à revisão como resultado
"Gostaria que fizéssemos code review como uma atividade de compartilhamento para o crescimento que podemos praticar agora mesmo, e como um meio de aumentar a produtividade por meio da melhoria da qualidade."
2 comentários
É necessária uma cultura que reconheça o tempo de revisão de código como parte do horário de trabalho.
Quando isso desce de cima para baixo com um cronograma extremamente apertado, não há muito o que fazer. Ou então quando definem uma due date e mandam tocar o trabalho.
No caso de empresas assim, a cultura geral tende a ser bem top-down.
Quando é algo que envolve ajustar só uma atividade, até dá para mudar com relativa facilidade, mas quando é o clima geral da organização, é difícil, porque depende de a liderança realmente querer isso.
Se não houver espaço para discutir a carga de trabalho, tente mudar a cultura passando pelo processo de convencer com base nas horas extras,
mas, se depois de um mês nada mudar, no fim das contas é mais eficiente procurar outra organização/empresa/setor.
Se você insistir além disso, acaba percebendo que está se desmanchando por dentro.
(Com “setor”, quero distinguir entre SI, startup e grande empresa. Tive a sensação de que o tipo de trabalho, a forma de trabalhar e a relação com a equipe são completamente diferentes.)