-
Notifications
You must be signed in to change notification settings - Fork 763
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
Add PyTzInfoAccess #2263
Add PyTzInfoAccess #2263
Conversation
fa4517f
to
bcbd4ae
Compare
0efd02b
to
e7326d2
Compare
e7326d2
to
bceacfd
Compare
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.
Hmm maybe to avoid the dance with Python None
we should just inline an equivalent of the FFI symbols.
bceacfd
to
67bfdc2
Compare
@pickfire You need to resolve conflicts otherwise CI won't run. |
67bfdc2
to
a1f97f1
Compare
… in datetime test.
23f0c10
to
0d0089e
Compare
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.
@davidhewitt I think this is good to merge now. Do you concur? If so, I'll merge and rebase the 0.16.4 release branch.
That would be great; thanks & I'll put the release live tonight 👍 |
Thank you for contributing to pyo3!
Please consider adding the following to your pull request:
PyO3's CI pipeline will check your pull request. To run its tests
locally, you can run
cargo xtask ci
. See its documentationhere.
As discussed in #1588 (comment)