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

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

Open
wants to merge 1 commit into
base: 3.13
Choose a base branch
from

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Nov 12, 2024

(cherry picked from commit feb3e0b)

Co-authored-by: Stephen Morton [email protected]

@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?

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