Básico
Code Review
Revisão do código de um Pull Request por outro membro da equipe para encontrar bugs, melhorar legibilidade e compartilhar conhecimento.
Muito além de "procurar erros"
O Code Review acontece dentro do Pull Request no GitHub. Tanto quem abre o PR quanto quem revisa aprende bastante nesse processo.
Checklist rápido antes de pedir revisão
- Você leu seu próprio diff no GitHub antes de marcar revisores?
- Removeu
dd(),console.log()de debug e arquivos temporários? - Os testes e o formatador de código passaram?
Como fazer bons comentários ao revisar
- Critique o código, nunca a pessoa.
- Explique o porquê da sugestão (ex.: legibilidade, segurança, performance) e, quando possível, mostre um exemplo curto de alternativa.
Exemplo Prático
markdown
Exemplo de comentário construtivo em um Pull Request:
> **Sugestão:** Aqui vale extrair essa verificação para uma função auxiliar ou usar early return para reduzir o aninhamento.
> Exemplo:
> ```php
> if (! $user->is_active) {
> return back()->withErrors(['email' => 'Conta inativa.']);
> }
> ```