Conversation
|
Hey there @marcelveldt, mind taking a look at this pull request as it has been labeled with an integration ( |
|
|
|
Looks like this will work for discovery 👍 |
|
@marcelveldt it looks like we currently don't do any error handling. Should we wrap aiohttp and aiohue errors in HomeAssitantErrors here: https://github.com/home-assistant/core/blob/dev/homeassistant/components/hue/bridge.py#L119 ? Or should we make another wrapper. |
|
Merging this so we can do the other in a follow-up PR. |
|
Opened #68396 |
Breaking change
Proposed change
Fixes #68388
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: