-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
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
CI: start testing Python 3.11 #47442
Conversation
Is this going to supersede #47032? |
Didn't realize there is an existing PR... |
You can just push to my PR directly if you want, or pull the changes here. It's a little farther down the pipeline, but I will not have time to work on it in the near future. |
|
FYI Windows will also fail until #47467 is merged in. |
Hello @fangchenli! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2022-08-17 19:19:05 UTC |
@fangchenli I've merged main into this PR for you just to see where we're at with the failures. I'm pretty close to finishing off #47467, and am going to try to get it merged this week. I think I have enough time where I'll be able to finish it up, hopefully before 3.11rc1. |
I accidentally pushed some commits here(I misconfigured github cli-meant to push to my own branch and pull request to this one), so I decided to go all the way to getting CI green. Sorry about the noise. At any rate, this is ready now pending #47467 merge. |
@@ -1,11 +1,3 @@ | |||
# This file is purposely frozen(does not run). DO NOT DELETE IT |
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.
I think it would be better to leave the comment in, and just comment out the if: false
below.
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.
Agreed. Even better to clarify
- "Freeze" if our dependencies don't support the latest python dev version
- "Unfreeze" if our dependencies support it, but the python dev version hasn't been officially released
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.
@fangchenli Do you have any objections here?
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.
@fangchenli Do you have any objections here?
No, I agree with you and @mroeschke
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.
Looks good. Just need to address #47442 (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.
LGTM pending green
@lithomas1 Thank you very much for finishing this up. |
Thanks for picking this up again. |
* CI: start testing Python 3.11 * fix * fix * fix macos * add error msg compat * fix error msg in test * xfail and skip tests * fix error * roll back * revert * maybe fix errors * update * try something * pin hypothesis * Don't check parentheses equality * typo * pin cython * correctly pin hypothesis * adjust test * remove compiler flag * typo'ed * fix tests * linter * clarify freeze/unfreeze instructions * whitespace * one more Co-authored-by: Thomas Li <[email protected]>
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.