Skip to content

bpo-34379: Move note for json.dump to function's docs#8730

Merged
methane merged 1 commit into
python:masterfrom
eallrich:note_update
Aug 11, 2018
Merged

bpo-34379: Move note for json.dump to function's docs#8730
methane merged 1 commit into
python:masterfrom
eallrich:note_update

Conversation

@eallrich

@eallrich eallrich commented Aug 10, 2018

Copy link
Copy Markdown
Contributor

I wasn't sure whether the versionchanged or note should go first. There's an example in the docs for datetime where versionchanged is first so I went with the same order.

Thank you for reviewing and please feel free to request changes as desired.

https://bugs.python.org/issue34379

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @eallrich for the PR, and @methane for merging it 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-8735 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 11, 2018
(cherry picked from commit 9e84084)

Co-authored-by: Evan Allrich <eallrich@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-8736 is a backport of this pull request to the 3.6 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 11, 2018
(cherry picked from commit 9e84084)

Co-authored-by: Evan Allrich <eallrich@gmail.com>
miss-islington added a commit that referenced this pull request Aug 11, 2018
(cherry picked from commit 9e84084)

Co-authored-by: Evan Allrich <eallrich@gmail.com>
methane pushed a commit that referenced this pull request Aug 11, 2018
(cherry picked from commit 9e84084)

Co-authored-by: Evan Allrich <eallrich@gmail.com>
@eallrich eallrich deleted the note_update branch August 11, 2018 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants