-
Notifications
You must be signed in to change notification settings - Fork 443
chore: Replace deprecated utcnow() #823
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
base: main
Are you sure you want to change the base?
Conversation
see warning in Python documentation: https://docs.python.org/3.9/library/datetime.html#datetime.datetime.utcnow
for more information, see https://pre-commit.ci
771e4f4 to
e3258b0
Compare
|
@willingc do you have time to take a look at this tiny change? it would safe us a few deprecation warnings in our logs 🙂 |
|
@MSeal do you have time to take a look at this? |
rgbkrk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. Not sure what's up with CI not reporting back.
e3258b0 to
54f3d76
Compare
|
I tried to trigger CI with another push but it doesn't seem to help 🤷 |
|
https://github.com/nteract/papermill/actions/runs/19180999485 displays
|
|
Odd that can't trigger it from here. When I get back to my laptop I'll change the repo settings to modern GitHub actions so we can kick this off and clean up the backlog of PRs. |
|
according to https://docs.github.com/en/actions/how-tos/manage-workflow-runs/approve-runs-from-forks, there should be an approve button here on the conversations tab though I'm not sure where or how it looks like |

What does this PR do?
replace
datetime.datetime.utcnow()withdatetime.datetime.now(datetime.timezone.utc)to resolve deprecation warningssee warning in Python documentation: https://docs.python.org/3.8/library/datetime.html#datetime.datetime.utcnow