-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
COMPAT: Drop reference to deprecated dateutil.zoneinfo.gettz #19006
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
Conversation
Original PR adding this was #9123, I believe it was just a bit conservative in switching over to the preferred API. |
thanks @pganssle yeah I think this will fix. not really sure why we had this older code. |
I could see someone writing code like this because they figured that |
@pganssle thanks. might as well just change it, prefer the logic to exist in dateutil anyhow. well that was the deprecation warning, so this patch should fix us. |
Codecov Report
@@ Coverage Diff @@
## master #19006 +/- ##
=========================================
Coverage ? 91.57%
=========================================
Files ? 150
Lines ? 48937
Branches ? 0
=========================================
Hits ? 44815
Misses ? 4122
Partials ? 0
Continue to review full report at Codecov.
|
thanks @pganssle |
I'm assuming these will be handled by CI:
git diff upstream/master -u -- "*.py" | flake8 --diff
Not sure this is necessary, it's a very "behind the scenes" fix, replacing
zoneinfo.gettz
with the superiortz.gettz()
: