[proxy] web.archive.org← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public

Conversation

Copy link
Member

serhiy-storchaka commented Oct 26, 2018

They return encoded bytes and are Python 2 artifacts.

https://bugs.python.org/issue33710

… module.

They return encoded bytes and are Python 2 artifacts.
Copy link
Member

warsaw left a comment

Choose a reason for hiding this comment

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

Looks great, but I have some wording suggestions, and one coding suggestion.

Copy link

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

warsaw and others added 6 commits October 26, 2018 21:55
Co-Authored-By: serhiy-storchaka <storchaka@gmail.com>
Co-Authored-By: serhiy-storchaka <storchaka@gmail.com>
Co-Authored-By: serhiy-storchaka <storchaka@gmail.com>
Co-Authored-By: serhiy-storchaka <storchaka@gmail.com>
Co-Authored-By: serhiy-storchaka <storchaka@gmail.com>
Copy link
Member Author

Thank you Barry! I have made the requested changes; please review again.

Copy link

Thanks for making the requested changes!

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature A feature request or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants