Como empresas de TI fazem revisão de código

• Motivos para fazer revisão de código:

  • Motivação: demonstrar profissionalismo para não prejudicar o revisor
  • Compartilhamento de conhecimento: o compartilhamento bidirecional de conhecimento eleva a capacidade técnica de toda a organização
  • Descoberta de erros: aumenta a possibilidade de encontrar erros com antecedência

• Google:

  • Todo código
  • Seguindo regras alinhadas às políticas internas
  • O menor possível, com frequência
  • Revisão em até 4 horas (revisões pequenas em até 1 hora, revisões grandes em até 5 horas)

• Microsoft

  • 39% fazem revisão pelo menos uma vez por dia

• BankSalad

  • Faz PRs pequenos com frequência
  • Para que o revisor entenda facilmente e revise rapidamente, criaram um template com mudanças, telas relacionadas e checklist para testes

• Woowa Brothers

  • Criaram e usam um template de MR
  • Usam um bot de MR para receber alertas sobre "que tipo de MR está chegando"

• Toss Payments

  • Às vezes adicionam o rótulo "Good Review" para usar expressões positivas

• Wemakeprice

  • Incentivam revisões detalhadas e o uso de emoticons

[kuber]

Microsoft: 39% das pessoas fazem revisão pelo menos uma vez por dia
...?
Não é muito pouco? Surpreendente.

[superwoou]

Acho que varia de empresa para empresa. Na nossa, todos os engenheiros fazem revisão pelo menos duas vezes por dia... haha

[ironlung]

Isso acaba sendo interpretado de forma diferente por cada pessoa que vê. ^^;