Repository navigation
Add missing intersphinx prefixes to cross-references - #1937
Closed
summeriver13 wants to merge 2 commits into
Closed
summeriver13 wants to merge 2 commits into
summeriver13 wants to merge 2 commits into
Conversation
Seven :ref: targets in the extension, clinic, and documentation pages resolved to nothing because their labels live in the CPython docs. Prefix them with the python: intersphinx name.
Member
Author
|
You're right, sorry — I misread the HTML: the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Seven
:ref:targets render as plain text instead of links: the labels live in the CPython docs and thepython:intersphinx prefix is missing.Checked on https://devguide.python.org — each shows up as
<span class="xref std std-ref">with nohref:developer-workflow/extension-modules/: "Stable ABI", "Limited API"development-tools/clinic/howto/: "Limited API", "heap type", "C "converter function""development-tools/clinic/: "Limited API" (under--limited)documentation/start-documenting/: "tracker"All seven targets already exist in the CPython docs, so only the prefix changes:
stable-abi,limited-c-apiDoc/c-api/stable.rst:54,:443heap-typesDoc/c-api/typeobj.rst:2586o_ampersandDoc/c-api/arg.rst:333reporting-bugsDoc/bugs.rst:1c-api.rstalready reaches the stability page viapython:stable. Withnitpicky = Truethese seven are currently "undefined label" warnings, tolerated only because--fail-on-warningis commented out in the Makefile.No wording changes, no new labels.
AI assistance: drafted with deepseek-v4.1-flash in the pi coding agent; each claim re-checked against the live site HTML and the CPython sources.