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

gh-101656: Fix "conversion from Py_ssize_t to int" warning in _testcapimodule #101657

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

sobolevn
Copy link
Member

@sobolevn sobolevn commented Feb 7, 2023

@sobolevn
Copy link
Member Author

sobolevn commented Feb 7, 2023

If the warning is gone, I hope that we can merge it :)

@sobolevn
Copy link
Member Author

sobolevn commented Feb 7, 2023

It is gone, tests do pass 🎉

@ambv ambv added the needs backport to 3.11 only security fixes label Feb 7, 2023
@ambv ambv merged commit acc2f3b into python:main Feb 7, 2023
@miss-islington
Copy link
Contributor

Thanks @sobolevn for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@sobolevn sobolevn added needs backport to 3.11 only security fixes and removed needs backport to 3.11 only security fixes labels Feb 8, 2023
@miss-islington
Copy link
Contributor

Thanks @sobolevn for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry, @sobolevn and @ambv, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker acc2f3b19d28d4bf3f8fb32357f581cba5ba24c7 3.11

sobolevn added a commit to sobolevn/cpython that referenced this pull request Feb 8, 2023
…ng in `_testcapimodule` (pythonGH-101657).

(cherry picked from commit acc2f3b)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@bedevere-bot
Copy link

GH-101680 is a backport of this pull request to the 3.11 branch.

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.

4 participants