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

CleanUp da solução #4

Open
rscouto opened this issue Jan 29, 2016 · 0 comments
Open

CleanUp da solução #4

rscouto opened this issue Jan 29, 2016 · 0 comments

Comments

@rscouto
Copy link
Contributor

rscouto commented Jan 29, 2016

  • Remoção de comentários desnecessários;
  • Adição de sumários(comentários e summary) e links de documentação da API;
  • Diminuir repetição;
  • Liberar recursos (IDisposables);
  • Tornar classes o mais readonly possível, para diminuir efeitos colaterais em cenários multi-thread;
  • Remove usings desnecessários;
  • User "var" ao invés de definir explicitamente o tipo;
  • Interpolar strings (C# 6);
  • Mover configuração da API para .config (versão, endpoint..etc, hoje só esta a apiKey)
  • Seguir padrão de case do .Net nas entidades e configurar no serializador a convenção a ser utilizada.
@rscouto rscouto modified the milestone: Versão 1.5 do Client Feb 4, 2016
rscouto added a commit that referenced this issue Feb 18, 2016
rscouto added a commit that referenced this issue Mar 2, 2016
rscouto added a commit that referenced this issue Mar 29, 2016
rscouto added a commit that referenced this issue Mar 29, 2016
rscouto added a commit that referenced this issue Mar 29, 2016
@rscouto rscouto removed this from the Versão 1.5 do Client milestone Jun 8, 2016
@rscouto rscouto added this to the Versão 2.x Client (vNext) milestone Dec 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant