Skip to content

Hopefully add Window / MacOS PyPy 3.10 / 3.11 support#12559

Merged
alex merged 2 commits intopyca:44.0.xfrom
alexandernst:44.0.x
Mar 7, 2025
Merged

Hopefully add Window / MacOS PyPy 3.10 / 3.11 support#12559
alex merged 2 commits intopyca:44.0.xfrom
alexandernst:44.0.x

Conversation

@alexandernst
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 🤔

- {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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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