Open freekode opened 1 week ago
Hey there @fabaff, @nzapponi, mind taking a look at this pull request as it has been labeled with an integration (openweathermap
) you are listed as a code owner for? Thanks!
I was hoping to be able to do another review pass on this before I'm flying all day tomorrow but it looks like the day got away from me. I'll be traveling for a week so I'll minimal time to look at this one. I'd like to try to get it merged before the 2024.6.x beta though given API 2.5 is going to go away.
Tested. Everything looks good.
Was able to switch from v2.5 to v3.0 and back.
I'm still traveling until Thursday, but I wanted to give you feedback on the issue so we can be sure to get it in before beta.
I'm still traveling until Thursday, but I wanted to give you feedback on the issue so we can be sure to get it in before beta.
Thanks, have a nice trip :)
I've marked this PR, as changes are requested that need to be processed. Please un-draft it once it is ready for review again by clicking the "Ready for review" button.
Thanks! 👍
../Frenck
Breaking change
Migration to new library with support of OneCall API (3.0)
Proposed change
Migrate new library
pyopenweathermap
I created specifically for Home Assistant. Currently, used pyowm library didn't have an update for a long time and seems dead. Remove forecast modes, one call allows to retrieve daily and hourly forecasts. Remove migration from V1 to V2 of config_flow, forecast mode isn't used anymore, there is no need for migration.New lib https://github.com/freekode/pyopenweathermap
Type of change
Additional information
Checklist
ruff format 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: