-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Hopefully add Window / MacOS PyPy 3.10 / 3.11 support #12559
Conversation
ee47bc3
to
ea3a1a3
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.
For the nightly issue, you can cherry pick 6606c83 to fix that.
.github/workflows/ci.yml
Outdated
- WINDOWS: {ARCH: 'x86', WINDOWS: 'win32'} | ||
PYTHON: {VERSION: "pypy-3.10", NOXSESSION: "tests-nocoverage"} | ||
- WINDOWS: {ARCH: 'x86', WINDOWS: 'win32'} | ||
PYTHON: {VERSION: "pypy-3.11", NOXSESSION: "tests-nocoverage"} |
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.
In general, backport branches should match what's on main for whatever they're backporting. In this case we don't test pypy on macOS and Windows, so that shouldn't be included 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.
Got it! I submitted a new patch!
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.
This part doesn't seem to have been removed.
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 don't see it in the diff 🤔
.github/workflows/ci.yml
Outdated
@@ -303,6 +305,8 @@ jobs: | |||
PYTHON: | |||
- {VERSION: "3.7", NOXSESSION: "tests-nocoverage"} | |||
- {VERSION: "3.13", NOXSESSION: "tests"} | |||
- {VERSION: "pypy-3.10", NOXSESSION: "tests-nocoverage"} | |||
- {VERSION: "pypy-3.11", NOXSESSION: "tests-nocoverage"} |
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.
It's still 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.
Oohh, sorry, I totally misunderstood you. I thought you were only talking about the chunk which prevented the CI from testing on winx86.
Fixed!
Don't exclude win_x86 . .
I'm not going to cut a release right away, unless there's an immediate need for this. |
This PR adds wheels for Windows and MacOS for PyPy 3.11.
I also took the liberty to improve a little bit the
exclude
section in the CI in order to spare some useless runs. 🙏There is 1 workflow that is failing, the linux+py3.12+rust-nightly, which I'm guessing is because of the rust-nightly (?).