Skip to content
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

Merged
merged 2 commits into from
Mar 7, 2025

Conversation

alexandernst
Copy link

@alexandernst alexandernst commented Mar 5, 2025

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 (?).

@alexandernst alexandernst force-pushed the 44.0.x branch 3 times, most recently from ee47bc3 to ea3a1a3 Compare March 5, 2025 22:45
Copy link
Member

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

- 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"}
Copy link
Member

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.

Copy link
Author

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!

Copy link
Member

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.

Copy link
Author

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 🤔

@@ -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"}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's still here.

Copy link
Author

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!

@alex
Copy link
Member

alex commented Mar 7, 2025

I'm not going to cut a release right away, unless there's an immediate need for this.

@alex alex merged commit 387ca00 into pyca:44.0.x Mar 7, 2025
99 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants