Checklist de apoio para aprovação de PR's

gistfile1.txt

Checklist para aprovação de um PR

Todos os code reviewers podem comentar diretamente na linha na qual foi encontrado o erro. Isso permite o time inteiro aprender com os erros e previni-los no futuro. É possível visualizar um code diff dentro do Bitbucket e ver o que foi modificado, aproveite!

[] - Este PR foi documentado nos documentos técnicos (ex. diagramas)
[] - Não tem código duplicado (Don't Repeat Yourself)
[] - Não existe redundância de componentes
[] - Código formatado corretamente
[] - Não existem comentários
[] - Não existe tentativa de modificar métodos de bibliotecas
[] - Não existem sujeiras de códigos como: (debug, print_r, exit, console.log, binding.pry, …)
[] - Variáveis, métodos, classes possuem nomes claros, dentro do contexto e que seguem a convenção
[] - Novas bibliotecas (gems, packages, …) foram instaladas nesse PR? Se sim, foi documentado?
[] - O design pattern foi implementado corretamente
[] - Segue o Single Responsibility Principle, na qual cada item tem apenas uma única responsabilidade e faz uma única coisa
[] - Não existe Hard coded como chaves, keys, textos sem i18n, if's com strings fixas, códigos comentados no HTML
[] - Possui testes unitários correspondentes
[] - Esse PR corresponde somente a uma história