Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove stray backtick in NEWS entry #115356

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

ezio-melotti
Copy link
Member

This removes a stray backtick that I noticed in a NEWS entry, introduced in #115213.

@hugovk, do we run sphinx-lint on NEWS entries?

@ezio-melotti ezio-melotti added type-bug An unexpected behavior, bug, or error skip issue skip news labels Feb 12, 2024
@bedevere-app bedevere-app bot added awaiting core review docs Documentation in the Doc dir labels Feb 12, 2024
@hauntsaninja hauntsaninja removed the type-bug An unexpected behavior, bug, or error label Feb 12, 2024
@hauntsaninja hauntsaninja enabled auto-merge (squash) February 12, 2024 19:23
@hauntsaninja hauntsaninja merged commit a82fbc1 into python:main Feb 12, 2024
31 checks passed
@hauntsaninja hauntsaninja added needs backport to 3.11 only security fixes needs backport to 3.12 bug and security fixes labels Feb 12, 2024
@miss-islington-app
Copy link

Thanks @ezio-melotti for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @ezio-melotti for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 12, 2024
(cherry picked from commit a82fbc1)

Co-authored-by: Ezio Melotti <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Feb 12, 2024

GH-115363 is a backport of this pull request to the 3.11 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 12, 2024
(cherry picked from commit a82fbc1)

Co-authored-by: Ezio Melotti <[email protected]>
@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Feb 12, 2024
@bedevere-app
Copy link

bedevere-app bot commented Feb 12, 2024

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

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Feb 12, 2024
hauntsaninja pushed a commit that referenced this pull request Feb 12, 2024
Remove stray backtick in NEWS entry (GH-115356)
(cherry picked from commit a82fbc1)

Co-authored-by: Ezio Melotti <[email protected]>
hauntsaninja pushed a commit that referenced this pull request Feb 12, 2024
Remove stray backtick in NEWS entry (GH-115356)
(cherry picked from commit a82fbc1)

Co-authored-by: Ezio Melotti <[email protected]>
@hugovk
Copy link
Member

hugovk commented Feb 12, 2024

@hugovk, do we run sphinx-lint on NEWS entries?

We do:

- repo: https://github.com/sphinx-contrib/sphinx-lint
rev: v0.9.1
hooks:
- id: sphinx-lint
args: [--enable=default-role]
files: ^Doc/|^Misc/NEWS.d/

But this sort of stray backtick isn't detected.

If I introduce one like this:

-Most exceptions are now ignored when attempting to set the ``__orig_class__``
+Most exceptions are now ignored when attempting to set the ``__orig_class__`

It is detected:

Sphinx Lint..............................................................Failed
- hook id: sphinx-lint
- exit code: 1

Misc/NEWS.d/next/Library/2024-02-09-07-20-16.gh-issue-115165.yfJLXA.rst:1: default role used (hint: for inline literals, use double backticks) (default-role)

@ezio-melotti ezio-melotti deleted the rm-stray-backtick branch February 12, 2024 21:09
@ezio-melotti
Copy link
Member Author

If it's not detected maybe we should create an issue about detecting it.

@ezio-melotti
Copy link
Member Author

There was an issue already:

fsc-eriker pushed a commit to fsc-eriker/cpython that referenced this pull request Feb 14, 2024
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 issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants