Checklists
Checklist — Code Review
Nenhum Pull Request é mesclado sem passar por este checklist. É o nosso contrato de qualidade.
1 min de leitura
Nenhum Pull Request é mesclado sem passar por este checklist. É o nosso contrato de qualidade.
Contrato de qualidade
- Código funcionando — testado localmente, sem erros conhecidos.
- Documentação atualizada — docs relevantes refletem a mudança.
- Testes — cobertura adequada para a mudança.
- Sem conflitos — branch atualizada com a base.
- Review — aprovado por ao menos um revisor.
- Deploy — pronto para seguir pelo pipeline.
Além do básico
O revisor também avalia:
- Simplicidade — não há solução mais simples; sem complexidade desnecessária.
- Padrões — segue os
standards/; commits no padrão. - Segurança — sem segredos no código; validação de entrada e permissões (RLS) corretas.
- Escalabilidade — nada que quebre ao crescer.
O objetivo do review não é aprovar rápido; é manter o padrão e elevar o time. Aponte problemas com clareza e sugira caminhos.