Weather units: accu to climacell (1/4)#60828
Weather units: accu to climacell (1/4)#60828rianadon wants to merge 3 commits intohome-assistant:devfrom
Conversation
|
Hey there @bieniu, mind taking a look at this pull request as it has been labeled with an integration ( |
|
Hey there @raman325, mind taking a look at this pull request as it has been labeled with an integration ( |
|
Hey there @Noltari, mind taking a look at this pull request as it has been labeled with an integration ( |
|
Hey there @mjj4791, @ties, @RobBie1221, mind taking a look at this pull request as it has been labeled with an integration ( |
|
I suggest that you do one integration at a time. The changes on each integration are more than just a single line of code... and with the new CI implementation it won't starve HA of build agents. |
Proposed change
Uses the new weather conversion in the weather entity ( #59533) to convert weather values to the configured units.
This particular PR involves some extra work to parse the units Accuweather gives. They might give back some speeds in knots, so I've added a
SPEED_KNOTSto the list of constants and conversion functions.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: