-
Notifications
You must be signed in to change notification settings - Fork 192
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
Upgrade pyo3 to 0.22 #1180
base: main
Are you sure you want to change the base?
Upgrade pyo3 to 0.22 #1180
Conversation
Hi @jelmer! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at [email protected]. Thanks! |
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Meta Open Source project. Thanks! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1180 +/- ##
=======================================
Coverage 91.26% 91.26%
=======================================
Files 261 261
Lines 26883 26883
=======================================
Hits 24535 24535
Misses 2348 2348 ☔ View full report in Codecov by Sentry. |
This looks good to me, thanks! I need some time before I can merge this to figure out what other things need upgrading to 0.22 internally at Meta |
When trying to build with this patch (and after refreshing
|
Newer versions of pyo3 warn about missing signatures
I've fixed the signature warning and have rebased to resolve merge conflicts. @mcepl the other warnings you're listing happen in "main" for me as well, they're not specific to this branch. |
Thank you, I will update the patch. It is weirdly nice to meet old acquaintance. So, Instagram? |
@zsol any news on this? |
Summary
This updates PyO3 to version 0.22, and adapt to the new APIs.
Test Plan