-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
Deprecate ISY994 custom cleanup entities service #85931
Conversation
Hey there @bdraco, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
c038873
to
f4f533b
Compare
We already have a service to reload a config entry so we can drop the |
I missed that somehow. I remembered the architecture discussion and thought it had stalled. Will deprecate. |
Breaking change
The
isy994.cleanup_entities
service is being deprecated and will be removed in2023.5.0
. Entities are automatically cleaned up when the integration is loaded or reloaded.Proposed change
Deprecate the
isy994.cleanup_entities
service is being deprecated and will be removed in2023.5.0
. Entities are automatically cleaned up when the integration is loaded or reloaded as of #85744), the custom service is no longer needed.Note: I am leaving the
isy994.reload
service for now--ultimately this can be removed if a core service is added to reload an integration from a service call.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
.To help with the load of incoming pull requests: