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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update xknx to 0.20.2 #69853
Update xknx to 0.20.2 #69853
Conversation
Hey there @Julius2342, @farmio, mind taking a look at this pull request as it has been labeled with an integration ( |
Should this go into a patch release? |
Yes, I tagged it |
The test that got adjusted in this PR is from #69697 which is not part of 2022.4.*. Do we need to tag this too since it's just tests or how do we proceed with that? If you just cherry-pick this it would most likely result in a failure. |
Proposed change
Bump XKNX to 0.20.2 and properly handle shutdown of devices.
Be aware: This was not an issue before as during reloading we would completely reinitialize the whole module.
But maybe we want to support dynamic entity loading at some point then this would have been bad.
https://github.com/XKNX/xknx/releases/tag/0.20.2 (for the diff just click diff to 0.20.1)
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: