Skip to content

Conversation

@skirpichev
Copy link
Member

@skirpichevskirpichev commented Nov 5, 2025

@bedevere-appbedevere-appbot mentioned this pull request Nov 5, 2025
@skirpichev
Copy link
MemberAuthor

Waiting for feedback in the issue thread. I'm planning soft-deprecate also Py_NAN.

@skirpichevskirpichev marked this pull request as ready for review November 6, 2025 03:21
@skirpichev
Copy link
MemberAuthor

I'll add docs here too.

Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ZeroIntensityZeroIntensity left a comment

Choose a reason for hiding this comment

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

I don't have much of an opinion here, I think this is fine. Would you mind creating a PR for <3.15 documenting this without the deprecation?

@ZeroIntensity
Copy link
Member

(Let's also wait on the C API WG before landing this one.)

@skirpichev
Copy link
MemberAuthor

Would you mind creating a PR for <3.15 documenting this without the deprecation?

See #141145.

@vstinnervstinner merged commit e202673 into python:mainNov 12, 2025
46 checks passed
@vstinner
Copy link
Member

Merged, thank you.

@skirpichevskirpichev deleted the soft-deprecate-Py_INFINITY/141004 branch November 12, 2025 12:47
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skirpichev@ZeroIntensity@vstinner