Code reviews: princípios
Este curso trata do julgamento por trás da revisão de código: quais comentários valem a pena e como decidir. Ele se apoia na documentação de práticas de engenharia do Google, que orienta aprovar uma mudança quando ela melhora a saúde geral do código, mesmo sem estar perfeita. As aulas mostram o que a CI deste app checa e o que sobra para uma pessoa, como rotular comentários como nit, opcional ou obrigatório e como manter o tom no código. Também tratam de tempo de resposta, escolha do veredicto, recebimento de feedback e uma comparação entre duas revisões do mesmo diff.
O que você vai aprender
- ✓O que a CI deste app checa e o que ela deixa para quem revisa
- ✓Rotular comentários como nit, opcional ou obrigatório
- ✓Escrever comentários sobre o código, e não sobre a pessoa
- ✓Escolher entre Approve, Request changes e Comment, e responder em até um dia útil
- ✓Separar discordância de confusão ao receber feedback
- ✓Comparar uma aprovação carimbada com uma revisão calibrada
Para que serve a revisão de código
A aula de abertura de "Pull requests" dá à revisão de código um tópico entre quatro: "um segundo par de olhos pega bugs, sugere abordagens melhores, e espalha conhecimento sobre a base de código pelo time". Aquele curso tinha sete aulas de mecânica do GitHub para cobrir: a visão de diff,…
Ler a aula completa grátis →Currículo
7 aulasPara que serve a revisão de código
Discute quais comentários valem a pena, a partir do padrão do Google de aprovar mudanças que melhoram o código.
O que a CI já checa, e o que ela deixa para uma pessoa
Mostra que a CI deste app roda Brakeman, bundler-audit, importmap audit e Rubocop, sem rodar nenhum teste.
Calibrando a gravidade: nit, opcional e obrigatório
Define os rótulos Nit e Optional da documentação do Google e por que comentários sem rótulo parecem obrigatórios.
Tom: comente o código, nunca a pessoa
Mostra por que o comentário deve falar do código e não do autor, e trata o ensino como parte da revisão.
O trabalho de quem revisa: tempo de resposta e escolha do veredicto
Trata da escolha entre Approve, Request changes e Comment, da resposta em um dia útil e do risco do carimbo.
Receber bem o feedback: a outra metade da revisão
Separa discordância de confusão do lado do autor e sugere levar conversas travadas para uma chamada rápida.
Juntando tudo: o mesmo diff, duas revisões
Compara um LGTM seco com uma revisão que pede um teste para lista vazia e rotula um nit.