Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Melhor organização das constantes do arquivo de generalConstants #9

Merged
merged 8 commits into from
Dec 2, 2018

Conversation

hugo-asb
Copy link
Collaborator

Mudanças propostas

O arquivo de GeneralConstants foi dividido em arquivos correlacionados. Também foram corrigidos alguns bugs.

Tipo de mudança

  • Bugfix (mudança que corrige um problema)
  • Nova funcionalidade (mudança que adiciona uma funcionalidade)
  • Mudança de funcionalidade (corrige ou adiciona recursos de uma funcionalidade que já existe)

Checklist

Marque as caixas que se aplicam ao pull request. Se você não tem certeza sobre algum deles, não hesite em perguntar. Nós estamos aqui para ajudar! Isto é simplesmente um lembrete do que vamos procurar antes de fundir seu código.

  • O nome do pull request deve ser significativo para representar a mudança que está sendo submetida.
  • Ter passado nos testes de integração.
  • Cobertura de testes não pode diminuir.
  • Os commits devem seguir a Política de Commits.
  • Marcar um revisor.
  • Linkar o pull request com a issue correspondente.

Anexos

Você pode colocar alguns anexos, como prints da tela, para exemplificar melhor a funcionalidade ou correção de bug que está sendo proposta.

Comentários adicionais

Caso você veja a necessidade de explicar mais a fundo as modificações propostas, use esta área para mostrar o porquê você escolheu essa solução, quais outras alternativas você considerou, alguma dúvida/comentário ou qualquer outro aspecto relevante do que foi feito. Não hesite em compartilhar seu conhecimento!

@hugo-asb hugo-asb changed the title Hugo Melhor organização das constantes do arquivo de generalConstants Nov 29, 2018
@andre-filho andre-filho merged commit 11a15bb into development Dec 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants