Skip to content

bpo-33710: Deprecate l*gettext() and related functions in the gettext module.#10139

Merged
serhiy-storchaka merged 7 commits intopython:masterfrom
serhiy-storchaka:deprecate-lgettext
Oct 27, 2018
Merged

bpo-33710: Deprecate l*gettext() and related functions in the gettext module.#10139
serhiy-storchaka merged 7 commits intopython:masterfrom
serhiy-storchaka:deprecate-lgettext

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

@serhiy-storchaka 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
Copy Markdown
Member

@warsaw 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.

Comment thread Doc/whatsnew/3.8.rst Outdated
Comment thread Doc/whatsnew/3.8.rst Outdated
Comment thread Doc/whatsnew/3.8.rst Outdated
Comment thread Doc/whatsnew/3.8.rst Outdated
Comment thread Lib/test/test_gettext.py
Comment thread Misc/NEWS.d/next/Library/2018-10-26-21-12-55.bpo-33710.Q5oXc6.rst Outdated
@bedevere-bot
Copy link
Copy Markdown

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

@serhiy-storchaka
Copy link
Copy Markdown
Member Author

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

@bedevere-bot
Copy link
Copy Markdown

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