KEMBAR78
GH-135763: AC: Use ``Py_ssize_t(allow_negative=False)`` by AA-Turner · Pull Request #138394 · python/cpython · GitHub
Skip to content

GH-135763: AC: Use Py_ssize_t(allow_negative=False) #138394

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 2, 2025
Merged

GH-135763: AC: Use Py_ssize_t(allow_negative=False) #138394

merged 2 commits into from
Sep 2, 2025

Conversation

@AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Sep 2, 2025 •
edited by bedevere-app bot
Loading

cc @picnixz, we should start using the new converter.

A

@AA-Turner AA-Turner requested a review from tiran as a code owner September 2, 2025 18:13
@bedevere-app bedevere-app bot mentioned this pull request Sep 2, 2025
Closed
picnixz
picnixz reviewed Sep 2, 2025
View reviewed changes
picnixz
picnixz reviewed Sep 2, 2025
View reviewed changes
Copy link
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

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

I like the fact that we now have a unified message most of the time.

@AA-Turner AA-Turner requested a review from picnixz September 2, 2025 19:42
@StanFromIreland StanFromIreland mentioned this pull request Sep 2, 2025
Closed
@wiomoc wiomoc mentioned this pull request Sep 2, 2025
Closed
@AA-Turner AA-Turner merged commit 98b4cd6 into python:main Sep 2, 2025
47 checks passed
@AA-Turner AA-Turner deleted the clinic/allow-negative branch September 2, 2025 20:29
lkollar pushed a commit to lkollar/cpython that referenced this pull request Sep 9, 2025
@AA-Turner @lkollar
pythonGH-135763: AC: Use Py_ssize_t(allow_negative=False) (python…
c0c2bba 
…#138394)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@picnixz picnixz picnixz approved these changes

@rhettinger rhettinger Awaiting requested review from rhettinger rhettinger is a code owner

@gpshead gpshead Awaiting requested review from gpshead gpshead is a code owner

@tiran tiran Awaiting requested review from tiran

Assignees

No one assigned

Labels

skip news topic-argument-clinic

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AA-Turner @picnixz