GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||
GeezerVison is showing github.com — GH-121970: Replace custom abstract method directive with the ``:abstract:`` option by AA-Turner · Pull Request #129311 · python/cpython Conversation
|
|
Current (https://docs.python.org/3/library/os.html#os.PathLike.__fspath__): Proposed (https://cpython-previews--129311.org.readthedocs.build/en/129311/library/os.html#os.PathLike.__fspath__): A |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Hrm... I weakly prefer the current rendering. The |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I suppose it's more of a design point than a question of maintenance burden. Sphinx's default Python domain should be good enough to document the Python documentation mostly using defaults. We will have some custom logic (e.g. for This is very likely to involve upstream changes though (made easier by the fact we now have decoupled our version support from downstream Linux distros). Is your preference to show e.g. "abstract method __fspath__()"? For methods in general we don't show the A |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Referring to it as "abstractmethod" or "abstract method" rather than simply "abstract" does make sense to me, because the decorator is |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I've opened sphinx-doc/sphinx#13271 on the Sphinx side. The three options we have are (1) keeping abstract, (2) using abstractmethod, or (3) using abstract method. The third has some symmetry with abstract properties, which are currently documented as abstract property. A |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
# Conflicts: # Doc/tools/extensions/pyspecific.py
|
Thanks @AA-Turner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Sorry, @AA-Turner, I could not cleanly backport this to |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Sorry, @AA-Turner, I could not cleanly backport this to |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…:abstract:`` option (python#129311) (cherry picked from commit 30e8924)
… the ``:abstract:`` option (pythonGH-129311) (cherry picked from commit 30e8924) Co-authored-by: Adam Turner <[email protected]>
|
GH-130440 is a backport of this pull request to the 3.12 branch. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
GH-130439 is a backport of this pull request to the 3.13 branch. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.