Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-39377: json: Remove encoding option.#18075

Merged
methane merged 2 commits into
python:masterpython/cpython:masterfrom
methane:remove-json-encodingmethane/cpython:remove-json-encodingCopy head branch name to clipboard
Jan 20, 2020
Merged

bpo-39377: json: Remove encoding option.#18075
methane merged 2 commits into
python:masterpython/cpython:masterfrom
methane:remove-json-encodingmethane/cpython:remove-json-encodingCopy head branch name to clipboard

Conversation

@methane

@methane methane commented Jan 20, 2020

Copy link
Copy Markdown
Member

Comment thread Misc/NEWS.d/next/Library/2020-01-20-13-00-35.bpo-39377.QSFdaU.rst Outdated
Co-Authored-By: Ammar Askar <ammar_askar@hotmail.com>
@methane methane merged commit 5492bfc into python:master Jan 20, 2020
@methane methane deleted the remove-json-encoding branch January 20, 2020 04:54
sthagen added a commit to sthagen/python-cpython that referenced this pull request Jan 20, 2020
bpo-39377: json: Remove the encoding option. (pythonGH-18075)
@ammaraskar

Copy link
Copy Markdown
Member

Oh sorry, I missed that there was a mention of encoding in https://github.com/python/cpython/blob/master/Doc/library/json.rst on line 286:

The other arguments have the same meaning as in :func:load, except
encoding which is ignored and deprecated since Python 3.1.

Should this be removed as well?

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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.