Content-Length: 325803 | pFad | http://github.com/python/cpython/pull/137207

9F [Doc] Remove unnecessary quotes from typing module by adorilson · Pull Request #137207 · python/cpython · GitHub
Skip to content

[Doc] Remove unnecessary quotes from typing module #137207

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

adorilson
Copy link
Contributor

@adorilson adorilson commented Jul 29, 2025

vstinner and others added 5 commits July 29, 2025 14:46
…#106335)

Remove private _PyThreadState and _PyInterpreterState C API
functions: move them to the internal C API (pycore_pystate.h and
pycore_interp.h). Don't export most of these functions anymore, but
still export functions used by tests.

Remove _PyThreadState_Prealloc() and _PyThreadState_Init() from the C
API, but keep it in the stable API.
This reverts commit ebfa093.
@StanFromIreland
Copy link
Member

This is a purely cosmetic change IMO, I see little benefit in removing the quotes. Have they caused confusion?

@adorilson
Copy link
Contributor Author

This is a purely cosmetic change IMO, I see little benefit in removing the quotes.

Beautiful is better than ugly. 🙂

In fact, this is purely cosmetic. But the more common form in docs is without these quotes (in a fast search).

Have they caused confusion?

It doesn´t cause confusion per si, in an absolute way or a concrete case for now. It can cause confusion, mainly because they are inside crases, both documentation maintainers and translators as well.

@JelleZijlstra
Copy link
Member

Aren't the quotes correct here because we're quoting the title of the work being referred to?

@adorilson
Copy link
Contributor Author

Aren't the quotes correct here because we're quoting the title of the work being referred to?

It could be the author's intention, but it isn't the pattern. If it were true, we should put more quotes everywhere. I don't think it's a good idea.

At the same module, we have

image

When the link is all over the title, the docs style cares about the title being highlighted. I think it is enough and quotes are unnecessary (and ugly and more complicated to maintain).

However, in the case of

“Type System Reference” section of the mypy docs

the link isn't about the title. I guess that is because the section hasn't been anchored in mypy docs (it is only a menu separator), so it is not possible to link it directly. Here, the quotes make sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

4 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/python/cpython/pull/137207

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy