AEX Engineering
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.

On this page