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
Add ColorMode enum to light platform #69223
Conversation
Hey there @home-assistant/core, mind taking a look at this pull request as it has been labeled with an integration ( |
Hey there @home-assistant/core, mind taking a look at this pull request as it has been labeled with an integration ( |
197c403
to
6f0b6d6
Compare
COLOR_MODE_COLOR_TEMP = "color_temp" | ||
COLOR_MODE_HS = "hs" | ||
COLOR_MODE_XY = "xy" | ||
COLOR_MODE_RGB = "rgb" | ||
COLOR_MODE_RGBW = "rgbw" | ||
COLOR_MODE_RGBWW = "rgbww" | ||
COLOR_MODE_WHITE = "white" # Must *NOT* be the only supported mode | ||
COLOR_MODE_WHITE = "white" | ||
|
||
VALID_COLOR_MODES = { |
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.
For a follow-up PR we might be able to get rid of this constant because whatever uses this constant should check if the enum has this value.
Breaking change
Developers only: All
COLOR_MODE_*
constants have been deprecated and replaced by aColorMode
enum.Proposed change
See above.
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: