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.

Sign up for GitHub

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

Jump to bottom

Remove FutureMixin class #33

Closed
Mr-Geekman opened this issue Aug 14, 2023 · 0 comments · Fixed by #58
Closed

Remove FutureMixin class #33

Mr-Geekman opened this issue Aug 14, 2023 · 0 comments · Fixed by #58
Assignees
@egoriyaa
Labels
enhancement New feature or request good first issue Good for newcomers priority/medium Medium priority task

Comments

@Mr-Geekman
Copy link

Issue by Mr-Geekman
Wednesday Jun 14, 2023 at 12:47 GMT
Originally opened as tinkoff-ai#1286

🚀 Feature Request

In current version we have etna.transforms.base.FutureMixin class. We doesn't need it anymore.

Proposal

  • Remove FutureMixin from all the classes there it is used.
  • Mark this class as deprecated and write that it will be removed in version 3.0.
    • Use deprecated package to mark FutureMixin.

Test cases

No response

Additional context

No response
@Mr-Geekman Mr-Geekman added enhancement New feature or request good first issue Good for newcomers priority/medium Medium priority task labels Aug 14, 2023
@egoriyaa egoriyaa self-assigned this Aug 24, 2023
@egoriyaa egoriyaa mentioned this issue Aug 24, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@egoriyaa egoriyaa

Labels
enhancement New feature or request good first issue Good for newcomers priority/medium Medium priority task
Projects
etna board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

delete FutureMixin etna-team/etna
2 participants
@Mr-Geekman @egoriyaa