8000 [3.13] gh-126699: allow AsyncIterator to be used as a base for Protocols (GH-126702) by miss-islington · Pull Request #126762 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

[3.13] 8000 gh-126699: allow AsyncIterator to be used as a base for Protocols (GH-126702) #126762

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

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

miss-islington
Copy link
Contributor
@miss-islington miss-islington commented Nov 12, 2024

(cherry picked from commit feb3e0b)

Co-authored-by: Stephen Morton github@tungol.org

…ls (pythonGH-126702)

(cherry picked from commit feb3e0b)

Co-authored-by: Stephen Morton <github@tungol.org>
@gpshead
Copy link
Member
gpshead commented Nov 13, 2024

This needs clear documentation as to the version constraints so that anyone attempting to rely on it knows specifically when it stopped working and in which patch releases it started working.

While its removal was a bug, this is also something that people can't reliably depend on in Python by simply knowing the major version if you backport it to 3.12 and 3.13...

@AlexWaygood
Copy link
Member

This needs clear documentation as to the version constraints so that anyone attempting to rely on it knows specifically when it stopped working and in which patch releases it started working.

While its removal was a bug, this is also something that people can't reliably depend on in Python by simply knowing the major version if you backport it to 3.12 and 3.13...

I'm not sure I agree. I feel like by the same logic, we'd never backport any bugfixes?

Using multiple inheritance with typing.Protocol and collections.abc.AsyncIterator currently raises an exception at runtime on Python 3.13, which is a bug. This change fixes that bug so that an exception is no longer raised. We don't typically go to great lengths in our docs to state when bugs like this were fixed, do we?

@Yhg1s
Copy link
Member
Yhg1s commented Dec 2, 2024

I agree with Alex, I think this is just a plain bugfix.

@Yhg1s Yhg1s merged commit a5672be into python:3.13 Dec 2, 2024
40 checks passed
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.

5 participants
0