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

Env --> Core #77

Open
6 tasks
alvarofpp opened this issue Aug 27, 2019 · 1 comment
Open
6 tasks

Env --> Core #77

alvarofpp opened this issue Aug 27, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@alvarofpp
Copy link
Member

Feature

Foi decidido renomear a classe Env para Core, como mudar parte do seu comportamento.

  • Tipo: Refatoração;
  • Branch: no final, deverá ser feito um merge para a branch version-2.

Checklist:

  • Implementar feature;
    • Renomear classe;
    • Retirar os métodos que não condizem com o comportamento de um Core. Colocar os métodos em uma outra classe, por enquanto;
  • Adicionar testes;
  • Atualizar docstring;
  • Atualizar documentação.

Para entender melhor

Atualmente a classe Env possui muitas funções, tornando o seu comportamento mais diverso do que deveria ser. Também há a questão do nome indicar algo local e não de algo que contêm as coisas para gerenciar o pacote corretamente. Devido a isso, a classe será remodelada para conter apenas o necessário para o seu funcionamento e as funcionalidades além disso serão atribuídas a outras classes (novas ou não).

Funcionalidades que não condizem com o comportamento do Core:

  • Métodos de print;
  • Métodos de requisições.
@alvarofpp alvarofpp added the enhancement New feature or request label Aug 27, 2019
@alvarofpp alvarofpp added this to the Versão 1.0.0 -> 2.0.0 milestone Aug 27, 2019
This was referenced Aug 27, 2019
@diegodiogenes
Copy link
Member

Concordo com a issue, principalmente no pensamento:

Também há a questão do nome indicar algo local e não de algo que contêm as coisas para gerenciar o pacote corretamente.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants