Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC Mark dev index as orphan - #34916

Merged
lesteve merged 1 commit into
scikit-learn:mainfrom
jeremiedbb:fix-dev-index-not-in-toc-tree
Sep 10, 2026
Merged

DOC Mark dev index as orphan#34916
lesteve merged 1 commit into
scikit-learn:mainfrom
jeremiedbb:fix-dev-index-not-in-toc-tree

Conversation

@jeremiedbb

Copy link
Copy Markdown
Member

(caught in the release branch #34905)

doc/developers/index.rst is only included in a toc tree when building the dev doc (so in PRs targeting main). Otherwise it's replaced by an external link so this page is not included in any toc tree, see

scikit-learn/doc/conf.py

Lines 1024 to 1029 in 2cc7893

# If development build, link to local page in the top navbar; otherwise link to the
# development version; see https://github.com/scikit-learn/scikit-learn/pull/22550
if parsed_version.is_devrelease:
development_link = "developers/index"
else:
development_link = "https://scikit-learn.org/dev/developers/index.html"

The issue only surfaces in PRs not targeting main and that modify this file (happens right now in the release PR), which explains why it didn't happen so far. Marking it as orphan says that it's fine if it's not in any toc tree, but still includes it in the toc tree when targeting main.

ping @lesteve

@jeremiedbb jeremiedbb added the To backport PR merged in master that need a backport to a release branch defined based on the milestone. label Sep 9, 2026
@lesteve
lesteve merged commit 98bcb4a into scikit-learn:main Sep 10, 2026
46 checks passed
lesteve pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 10, 2026
@lesteve lesteve mentioned this pull request Sep 10, 2026
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation To backport PR merged in master that need a backport to a release branch defined based on the milestone.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants