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

Replace dataclasses with attrs and slotted classes #976

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nolar
Copy link
Owner

@nolar nolar commented Nov 13, 2022

In preparation for further refactoring for proper time handling — so that this single change would be tested in all Python versions.

Besides, attrs produces the slotted classes by default, which is good for memory footprint, especially on the frequently used classes like HandlerState, Outcome, etc.

In all other aspects, it is essentially a drop-in replacement with some minor changes.

The attrs dependency is already installed for one of the sub-dependencies, so nothing new is essentially added to the deployment.

@nolar nolar added the refactoring Code cleanup without new features added label Nov 13, 2022
@nolar nolar force-pushed the dataclasses-to-attrs branch 2 times, most recently from df3a655 to 71820f9 Compare November 13, 2022 21:59
@nolar nolar force-pushed the dataclasses-to-attrs branch from 71820f9 to f21c018 Compare November 13, 2022 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Code cleanup without new features added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant