Checklist para de verificação de PR

Antes de enviar a sua tarefa para code review, verifique os seguintes pontos:

• [ ] Utiliza buscas na BLL
  • [ ] Campos buscando via Ide e não código
  • [ ] Verificar integridade do SQL (se realmente precisa das informações que você está colocando)
• [ ] Utilizar métodos de extensão
• [ ] Possui condicionais
  • [ ] Evitar condicional negativa
  • [ ] Incluir parênteses em verificação com mais de uma condição
  • [ ] Utilizar .Equals ao invés de ==
  • [ ] Caso tenha múltiplas condicionais, separar elas por linha
• [ ] Foram criadas propriedades ou objetos novos
  • [ ] Inicializar sempre as propriedades
  • [ ] Não colocar como nulo
• [ ] Manipula componentes do DevExpress
  • [ ] Utilizar propriedades EditText, EditValue
  • [ ] Recuperar valores com métodos de extensão
  • [ ] Utilizar apenas o .Value de checkbox
• [ ] Foram criadas variáveis
  • [ ] Utilizar o camelCase para nomenclatura
  • [ ] Forçar a tipagem das variáveis (não utilizar var para declaração)
• [ ] Foram criados métodos
  • [ ] Possui parâmetros opcionais
    • [ ] Todas as chamadas de método devem ter o parâmetro explícito
• [ ] Possui palavras ou frases que podem ser aprimorados
  • [ ] Consultar no projeto ConstantManager
  • [ ] Verificar nas propriedades da classe se possui alguma função que pega o que deseja
• [ ] Impacto das mudanças afeta mais de um modulo do sistema
  • [ ] Verificar mudanças no ETrade
  • [ ] Verificar mudanças no PDV
  • [ ] Verificar mudanças no Gourmet
  • [ ] Verificar sincronia Bridge
• [ ] Se for tornar uma função obsoleta
  • [ ] Incluir a tag [Obsolete(“migrado para ….”)]
  • [ ] Renomear a função para oldNomeDaFuncao