Skip to content

bpo-39019: Implement missing __class_getitem__ for http.cookies.Morsel #17563

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.

Sign up for GitHub

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

Jump to bottom
Closed
isidentical wants to merge 4 commits into from
Closed

bpo-39019: Implement missing __class_getitem__ for http.cookies.Morsel #17563

isidentical wants to merge 4 commits into from

Conversation

@isidentical
Copy link
Member

@isidentical isidentical commented Dec 10, 2019 •
edited by bedevere-bot
Loading

pablogsal
pablogsal reviewed Dec 10, 2019
View reviewed changes
@ilevkivskyi
Copy link
Member

ilevkivskyi commented Dec 27, 2019

@isidentical please don't force-push, because the reviewers will not get a notification from GitHub.

@pablogsal I think you can look at this again.

@isidentical
Copy link
Member Author

isidentical commented Dec 27, 2019

I see, thanks for advice @ilevkivskyi

ilevkivskyi
ilevkivskyi reviewed Dec 27, 2019
View reviewed changes
@isidentical isidentical mentioned this pull request Feb 23, 2020
Merged
@isidentical
Copy link
Member Author

isidentical commented Apr 6, 2020

Closing since pep 585 accepted

@isidentical isidentical closed this Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pablogsal pablogsal pablogsal left review comments

@ilevkivskyi ilevkivskyi ilevkivskyi left review comments

Assignees

No one assigned

Labels

awaiting review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@isidentical @ilevkivskyi @pablogsal @the-knights-who-say-ni @bedevere-bot