Skip to content

Conversation

@winniehell
Copy link

@winniehell winniehell commented Aug 27, 2025

What does this PR do?

replace datetime.datetime.utcnow() with datetime.datetime.now(datetime.timezone.utc) to resolve deprecation warnings

see warning in Python documentation: https://docs.python.org/3.8/library/datetime.html#datetime.datetime.utcnow

@winniehell winniehell marked this pull request as ready for review August 27, 2025 10:09
@winniehell winniehell force-pushed the winniehell-replace-utcnow branch from 771e4f4 to e3258b0 Compare August 27, 2025 10:28
@winniehell
Copy link
Author

@willingc do you have time to take a look at this tiny change? it would safe us a few deprecation warnings in our logs 🙂

@winniehell
Copy link
Author

@MSeal do you have time to take a look at this?

Copy link
Member

@rgbkrk rgbkrk left a 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.

@winniehell winniehell force-pushed the winniehell-replace-utcnow branch from e3258b0 to 54f3d76 Compare November 7, 2025 20:56
@winniehell
Copy link
Author

I tried to trigger CI with another push but it doesn't seem to help 🤷

@winniehell
Copy link
Author

https://github.com/nteract/papermill/actions/runs/19180999485 displays This workflow is awaiting approval from a maintainer in https://github.com/nteract/papermill/pull/823 for me:

grafik

@rgbkrk
Copy link
Member

rgbkrk commented Nov 7, 2025

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.

@winniehell
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants