Skip to content

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 1, 2025

(cherry picked from commit 6528cf4)

Co-authored-by: Marc Mueller [email protected]
Co-authored-by: Adam Turner [email protected]

…-137864)

(cherry picked from commit 6528cf4)

Co-authored-by: Marc Mueller <[email protected]>
Co-authored-by: Adam Turner <[email protected]>
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM

@cdce8p
Copy link
Contributor

cdce8p commented Sep 2, 2025

Thanks @vstinner! Looks like this one is ready for merge now, all checks have passed.

@vstinner
Copy link
Member

vstinner commented Sep 2, 2025

Thanks @vstinner! Looks like this one is ready for merge now, all checks have passed.

This fix will have to wait for Python 3.14.1. It's not special enough to fit into Python 3.14.0 final release.

@cdce8p
Copy link
Contributor

cdce8p commented Sep 2, 2025

Thanks @vstinner! Looks like this one is ready for merge now, all checks have passed.

This fix will have to wait for Python 3.14.1. It's not special enough to fit into Python 3.14.0 final release.

Ahh, I see. Tbh I had hoped it could still be included as it fixes a regression introduced in 6146295 (3.14.0a7). Without the fix I'll need to patch our alpine docker builds for 3.14.0.

@vstinner
Copy link
Member

vstinner commented Sep 2, 2025

cc @hugovk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants