Skip to content

gh-136216: Update JSON RFC references from 7159 to 8259#144453

Closed
kovan wants to merge 1 commit intopython:mainfrom
kovan:gh-136216-json-rfc-update
Closed

gh-136216: Update JSON RFC references from 7159 to 8259#144453
kovan wants to merge 1 commit intopython:mainfrom
kovan:gh-136216-json-rfc-update

Conversation

@kovan
Copy link
Copy Markdown

@kovan kovan commented Feb 3, 2026 •
edited by github-actions bot
Loading

Summary

  • Updates JSON module documentation to reference RFC 8259 instead of RFC 7159
  • RFC 8259 obsoleted RFC 7159 in December 2017
  • Updates the errata footnote link accordingly

Test plan

  • make check passed
  • Documentation builds correctly

🤖 Generated with Claude Code

📚 Documentation preview 📚: https://cpython-previews--144453.org.readthedocs.build/

@bedevere-app bedevere-app bot added awaiting review docs Documentation in the Doc dir skip news labels Feb 3, 2026
@bedevere-app bedevere-app bot mentioned this pull request Feb 3, 2026
Open
@github-project-automation github-project-automation bot moved this to Todo in Docs PRs Feb 3, 2026
picnixz
picnixz requested changes Feb 3, 2026
View reviewed changes
@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Feb 3, 2026

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@kovan @claude
pythongh-136216: Update JSON RFC references from 7159 to 8259
0dbf920 
RFC 8259 obsoleted RFC 7159 in December 2017. Update the json module
documentation to reference the current RFC.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@kovan
Copy link
Copy Markdown
Author

kovan commented Feb 3, 2026

I have made the requested changes; please review again

@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Feb 3, 2026

Thanks for making the requested changes!

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

@bedevere-app bedevere-app bot requested a review from picnixz February 3, 2026 23:52
@picnixz
Copy link
Copy Markdown
Member

picnixz commented Feb 4, 2026

If my comments are not addressed (by replying to them / changing code), please don't request a review.

@kovan
Copy link
Copy Markdown
Author

kovan commented Feb 27, 2026

I have made the requested changes; please review again

@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Feb 27, 2026

Thanks for making the requested changes!

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

@picnixz
Copy link
Copy Markdown
Member

picnixz commented Feb 28, 2026

Considering the author is now restricted, I'm closing their corresponding PRs.

@picnixz picnixz closed this Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@picnixz picnixz Awaiting requested review from picnixz

Assignees

No one assigned

Labels

awaiting change review docs Documentation in the Doc dir skip news

Projects

Docs PRs
Status: Todo

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kovan @picnixz