Skip to content

Conversation

@smontanaro
Copy link
Contributor

@smontanarosmontanaro commented Jan 31, 2024

As @GPHemsley pointed out, #29469 omitted versionadded notes for the 2 new items.


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

@GPHemsley pointed out that with the recent changes in python#29469, two new data items failed to get `versionadded` notes.
@merwokmerwok added the needs backport to 3.12 only security fixes label Jan 31, 2024
@terryjreedyterryjreedy changed the title Add versionadded notes for QUOTE_NOTNULL and QUOTE_STRINGSgh-67230: Add versionadded notes for QUOTE_NOTNULL and QUOTE_STRINGSFeb 1, 2024
@terryjreedyterryjreedy merged commit 586057e into python:mainFeb 1, 2024
@miss-islington-app
Copy link

Thanks @smontanaro for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 1, 2024
…RINGS (pythonGH-114816) As @GPHemsley pointed out, pythonGH-29469 omitted `versionadded` notes for the 2 new items. (cherry picked from commit 586057e) Co-authored-by: Skip Montanaro <skip.montanaro@gmail.com>
@bedevere-app
Copy link

GH-114840 is a backport of this pull request to the 3.12 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.12 only security fixes label Feb 1, 2024
@terryjreedy
Copy link
Member

In my view, a correction to a defective patch should be attached to the issue for the original patch (unless there is a new issue, which there is not here). Merging.

terryjreedy pushed a commit that referenced this pull request Feb 1, 2024
…TRINGS (GH-114816) (#114840) As @GPHemsley pointed out, GH-29469 omitted `versionadded` notes for the 2 new items. (cherry picked from commit 586057e) Co-authored-by: Skip Montanaro <skip.montanaro@gmail.com>
@smontanarosmontanaro deleted the patch-8 branch February 1, 2024 15:07
@smontanaro
Copy link
ContributorAuthor

In my view, a correction to a defective patch should be attached to the issue for the original patch (unless there is a new issue, which there is not here). Merging.

Thanks. It didn't occur to me to amend a closed issue.

aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…RINGS (python#114816) As @GPHemsley pointed out, python#29469 omitted `versionadded` notes for the 2 new items.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docsDocumentation in the Doc dirskip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@smontanaro@terryjreedy@merwok@Eclips4