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
Add device_class safety to synology_dsm storage binary_sensors #39310
Conversation
Hey there @ProtoThis, @Quentame, mind taking a look at this pull request as its been labeled with an integration ( |
Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
thanks @MartinHjelmare I had copied this from the Stookalert integration |
Please do that in a separate PR. |
from homeassistant.components.binary_sensor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
ok, changed and imported device_class Safety, compare :
|
please advise what to do. Isort changes to
but also
which was untouched by me... which hardly seems correct? Don't want to keep creating new error messages here. |
Make sure to run isort in the project directory so that our isort config is picked up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
asper discussion here #39310 (comment)
…assistant#39310) * add device_class: safety to storage binary_sensors * Update binary_sensor.py * Update homeassistant/components/synology_dsm/binary_sensor.py Co-authored-by: Martin Hjelmare <marhje52@gmail.com> * Update binary_sensor.py * Import device_class Safety from homeassistant.components.binary_sensor * Update binary_sensor.py Co-authored-by: Martin Hjelmare <marhje52@gmail.com>
Hey @MartinHjelmare shouldn't we change this
|
Yes, please. |
Breaking change
Proposed change
Changes the state and icon to reflect the safety state this binary_sensor attempts to reflect.
Type of change
Example entry for
configuration.yaml
:# Example configuration.yaml
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: