-
Notifications
You must be signed in to change notification settings - Fork 59
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
Bump pyowm to latest version #157
Labels
Comments
Hey there, Chris is working on a big update to this Skill at the moment - including updating the pyowm API version 👍 |
5 tasks
seems requirements.txt kept untouched |
Hey @chrisveilleux was this meant to get bumped with the move to the "One Call API"? |
Confusingly i haven't found a bit we're using from pyowm |
I removed pyowm from the skill, but forgot to remove the dependency. A PR 169 addresses this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Weather skill's
requirements.txt
is currently fixed topyowm==2.6.1
while pyowm is now at version 3.1. OWM introduced some interesting new features that are supported by pyowm 3.x, e.g. One Call APIIn case there are specific reasons to stay with pyowm 2.x a version bump to 2.10-LTS should be considered.According to this timeline the pyowm 2.x long-term support will end 05/30/2021 (given that pyowm 3.0 was released on 05/30/2020)The text was updated successfully, but these errors were encountered: