1. ⭐Crie um processo melhor: para ajudar seu time a entregar um software melhor e mais rápido.
2. 📈Faça melhores code reviews: que impulsionam a qualidade do código e eleva as skills de seus colegas.
3. ✅Escreva um código melhor: em que é aprovado na primeira review.
https://google.github.io/eng-practices/review/reviewer/comments.html#courtesy
https://dropbox.github.io/dbx-career-framework/ic2_software_engineer.html
Capacite seu time a entregar um software de alta qualidade, mais rápido
💬 "LGTM! 👍" [3 segundos depois]
💬 "Esse código é horrível. Não tem como realizarmos a manutenção do que você escreveu."
💬 "Esse método é ilegível, nós devemos refatorar ele."
https://github.com/renangabriel27/code-review-guideline
Impulsionar a qualidade do código e elevar as skills do time
❌ "Você não checou o valor null."
✅ "Esse valor do input pode ser null, causando um erro de servidor. Se o valor for null, você poderia lançar um erro client/side."
Ou
✅ "O que poderia acontecer se for setado um valor null?"
❌ "Mude o nome da variável."
✅ "Mudança, não bloqueante:
Eu recomendaria mudar o nome da variável de `purchaseStatus` para `orderStatus`. A variável armazena o status de um pedido, não a compra."
❌ "Use o Promise.all"
✅ "Promise.all nesse caso de várias requisições vai rodar todas elas de forma assíncrona e finalizar somente depois de todas terem rodado, otimizando a velocidade do carregamento da página"
❌ "Use o Promise.all"
✅ "Promise.all nesse caso de várias requisições vai rodar todas elas de forma assíncrona e finalizar somente depois de todas terem rodado, otimizando a velocidade do carregamento da página.
Para mais contexto: https://dev.to/cristuker/javascript-porque-usar-promiseall-4jc3"
Master the Code Review (Course)
Dropbox career framework (Site)
There’s a human on the other side of your code review by Tadas Antanavicius (Post)
Implementing a Strong Code-Review Culture by Derek Prior (Talk)
Clean Code como tornar nossos códigos legíveis para seres humanos (Slides)
Code Review Guidelines - (Repositório/Github)