◐ Shell
reader mode source ↗
Skip to content

gh-104873: Add typing.get_protocol_members and typing.is_protocol#104878

Merged
JelleZijlstra merged 14 commits into
python:mainfrom
JelleZijlstra:protoattrs
Jun 14, 2023
Merged

gh-104873: Add typing.get_protocol_members and typing.is_protocol#104878
JelleZijlstra merged 14 commits into
python:mainfrom
JelleZijlstra:protoattrs

Conversation

@JelleZijlstra

@JelleZijlstra JelleZijlstra commented May 24, 2023 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

@bedevere-bot bedevere-bot added the label May 24, 2023
@bedevere-bot bedevere-bot mentioned this pull request May 24, 2023
Closed
@bedevere-bot

bedevere-bot commented May 24, 2023

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@JelleZijlstra JelleZijlstra changed the title gh-104873: Add typing.get_protocol_members May 24, 2023
@JelleZijlstra

JelleZijlstra commented May 24, 2023

Copy link
Copy Markdown
Member Author

I didn't expect the Spanish inquisition!

@bedevere-bot

bedevere-bot commented May 24, 2023

Copy link
Copy Markdown

Nobody expects the Spanish Inquisition!

@AlexWaygood: please review the changes made to this pull request.

@bedevere-bot bedevere-bot requested a review from AlexWaygood May 24, 2023 18:04
JelleZijlstra and others added 2 commits May 24, 2023 13:10
@JelleZijlstra @AlexWaygood
Apply suggestions from code review …
d1e9dec 
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@JelleZijlstra
Merge branch 'main' into protoattrs
4f3e627
@JelleZijlstra @AlexWaygood
Update Doc/whatsnew/3.13.rst …
c91acbb 
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
AlexWaygood
AlexWaygood approved these changes Jun 5, 2023
View reviewed changes

@AlexWaygood AlexWaygood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

LGTM, but I'd love to get a third opinion from @gvanrossum or @Fidget-Spinner on the API, if possible

16 hidden items Load more…
@JelleZijlstra

JelleZijlstra commented Jun 5, 2023

Copy link
Copy Markdown
Member Author

FYI I also just emailed typing-sig about this to ask for feedback.

@JelleZijlstra

JelleZijlstra commented Jun 13, 2023

Copy link
Copy Markdown
Member Author

Nobody spoke up against this on typing-sig, time to merge?

JelleZijlstra and others added 3 commits June 13, 2023 19:37
@JelleZijlstra @AlexWaygood
Apply suggestions from code review …
fc4eb5d 
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@JelleZijlstra
Merge remote-tracking branch 'upstream/main' into protoattrs
7e8636a
@JelleZijlstra @AlexWaygood
Update Lib/typing.py …
8b31c81 
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
AlexWaygood
AlexWaygood approved these changes Jun 14, 2023
View reviewed changes

@AlexWaygood AlexWaygood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

Time to merge, in my opinion!

@JelleZijlstra JelleZijlstra merged commit fc8037d into python:main Jun 14, 2023
@JelleZijlstra JelleZijlstra deleted the protoattrs branch June 14, 2023 12:43
@JelleZijlstra JelleZijlstra restored the protoattrs branch September 10, 2024 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexWaygood AlexWaygood AlexWaygood approved these changes

@gvanrossum gvanrossum Awaiting requested review from gvanrossum

@Fidget-Spinner Fidget-Spinner Awaiting requested review from Fidget-Spinner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JelleZijlstra @bedevere-bot @AlexWaygood