FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Allow Protocols to inherit from typing_extensions.Buffer or collections.abc.Buffer by AlexWaygood · Pull Request #220 · python/typing_extensions · GitHub

Allow Protocols to inherit from typing_extensions.Buffer or collections.abc.Buffer - #220

Merged
srittau merged 4 commits into
python:mainfrom
AlexWaygood:buffer-subclassing
Jun 5, 2023
Merged

Allow Protocols to inherit from typing_extensions.Buffer or collections.abc.Buffer#220
srittau merged 4 commits into
python:mainfrom
AlexWaygood:buffer-subclassing

Conversation

Copy link
Copy Markdown
Member

Backport of python/cpython#104827

AlexWaygood marked this pull request as draft June 4, 2023 16:40
AlexWaygood marked this pull request as ready for review June 4, 2023 16:50
AlexWaygood changed the title Allow Protocols to inherit from typing_extensions.Buffer Allow Protocols to inherit from typing_extensions.Buffer or collections.abc.Buffer Jun 4, 2023
Comment on lines +2740 to +2744
@skipUnless(
hasattr(collections.abc, "Buffer"),
"needs collections.abc.Buffer to exist"
)
@skip_if_py312b1

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This test isn't run in CI at all right now, but I've confirmed locally that it passes with the CPython main branch (so when 3.12b2 comes along, it should pass in CI for us here)

srittau merged commit afabbbd into python:main Jun 5, 2023
AlexWaygood deleted the buffer-subclassing branch June 5, 2023 09:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL