-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
[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
base: main
Are you sure you want to change the base?
Conversation
…#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.
This is a purely cosmetic change IMO, I see little benefit in removing the quotes. Have they caused confusion? |
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).
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. |
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 ![]() 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. |
📚 Documentation preview 📚: https://cpython-previews--137207.org.readthedocs.build/