-
Notifications
You must be signed in to change notification settings - Fork 33
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
Replace pytz with zoneinfo. Fix for #222 #225
Conversation
Replace pytz with zoneinfo
Replace pytz with zoneinfo
@dbiczo thanks for this PR. For some reason I had not even been getting these warnings (probably because I Only use it for forecast information). I assume you have fully tested that it doesn't break anything else. It looks like a fairly trivial change. |
Yes, tested over a full day. Havn't tested setup flow but don't think it will affect that. |
Awesome. Thanks I will merge and do a release. |
I did the release and it broke for me so I have removed it for now. It looks like there is a typo in the fix you submitted. I will edit on main and retry. Actually I think line 8 should simply be removed since you added the import needed on line 18. Do you agree? |
Thanks. Sorry for the typo. I tested live on my setup and copied the changes to the pull request. |
I was copying what was there for pytz and both lines were there. |
Yep, they both need to be there. I did a quick test without the first one and got errors. Will do a new release shortly. |
Release 1.3.2 is done and gives me no errors. |
Great. Thanks for you help. |
That commit introduce error #226 since we are no longer using pytz, but pytz.timezone is still referenced for the UV text |
Fix "Detected blocking call to open inside the event loop" warnings by replacing pytz with zoneinfo.