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

Make dataclasses frozen #482

Closed
wants to merge 1 commit into from

Conversation

canton7
Copy link
Collaborator

@canton7 canton7 commented Dec 23, 2023

HA is moving to a model where dataclasses need to be frozen (to aid caching), and will start logging depreciation warnings in 2024.x. Let's get ahead of the game.

See home-assistant/core#105211

HA is moving to a model where dataclasses need to be frozen (to aid
caching), and will start logging depreciation warnings in 2024.x. Let's
get ahead of the game.

See home-assistant/core#105211
@canton7 canton7 force-pushed the feature/dataclass-freeze branch from 9076a5f to e31a916 Compare December 23, 2023 21:58
@canton7
Copy link
Collaborator Author

canton7 commented Dec 23, 2023

Hmph, looks like we can only pass frozen=True if the base class is frozen. This probably depends on #480 then.

@canton7 canton7 closed this Dec 24, 2023
@canton7 canton7 deleted the feature/dataclass-freeze branch December 24, 2023 12:22
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.

1 participant