Skip to content

Conversation

@tpoliaw
Copy link
Contributor

@tpoliaw tpoliaw commented Oct 19, 2025

PyIterator::send is a public method but the returned values are not
usable while PySendResult is not public. Making it public allows
callers to match on the result and access the wrapped values.

PyIterator::send is a public method but the returned values are not
usable while `PySendResult` is not public. Making it public allows
callers to match on the result and access the wrapped values.
@tpoliaw
Copy link
Contributor Author

tpoliaw commented Oct 19, 2025

Not sure of the usual workflow here, would you prefer multiple commits to be squashed and rebased in the PR or are they all squashed and merged if/when the PR is closed and individual commits are easier to review?

@bschoenmaeckers
Copy link
Member

bschoenmaeckers commented Oct 20, 2025

Tnx!

Not sure of the usual workflow here, would you prefer multiple commits to be squashed and rebased in the PR or are they all squashed and merged if/when the PR is closed and individual commits are easier to review?

This is fine. It will get squashed on merge anyway.

@davidhewitt davidhewitt added this pull request to the merge queue Oct 20, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 20, 2025
@tpoliaw
Copy link
Contributor Author

tpoliaw commented Oct 20, 2025

Using quay.io/pypa/manylinux2014_x86_64:latest Docker image
/usr/bin/docker pull quay.io/pypa/manylinux2014_x86_64:latest
Error response from daemon: received unexpected HTTP status: 504 Gateway Time-out

This looks like a transient failure unrelated to the change

@LilyFirefly LilyFirefly added this pull request to the merge queue Oct 20, 2025
@alex
Copy link
Contributor

alex commented Oct 20, 2025

Almost certainly caused by this morning's AWS incident.

@alex alex mentioned this pull request Oct 20, 2025
2 tasks
Merged via the queue into PyO3:main with commit 67ac7ab Oct 20, 2025
43 checks passed
@tpoliaw tpoliaw deleted the public_send_result branch October 20, 2025 12:54
davidhewitt pushed a commit that referenced this pull request Oct 20, 2025
* Expose types::iterator::PySendResult in types module

PyIterator::send is a public method but the returned values are not
usable while `PySendResult` is not public. Making it public allows
callers to match on the result and access the wrapped values.

* Gate export of PySendResult behind Py_3_10

* Document PySendResult and add newsfragment
davidhewitt pushed a commit that referenced this pull request Oct 21, 2025
* Expose types::iterator::PySendResult in types module

PyIterator::send is a public method but the returned values are not
usable while `PySendResult` is not public. Making it public allows
callers to match on the result and access the wrapped values.

* Gate export of PySendResult behind Py_3_10

* Document PySendResult and add newsfragment
Sign up for free to 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.

4 participants