Skip to content

docs: render markdown headings relative to their section - #397

Open
yeoldegrove wants to merge 2 commits into
mainfrom
docs/gldq
Open

yeoldegrove wants to merge 2 commits into
mainfrom
docs/gldq

Conversation

@yeoldegrove

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

sphinx-markdown-builder pins rubric and option/signature headings to level 3 while section titles use their absolute nesting depth. Nested sphinx-click and sphinxcontrib-autoprogram sections therefore jump from level 3 to 5 in the generated markdown, which the aggregated docs reject (gldq HEAD-SKIP).

Override the markdown translator so those headings are rendered one level below the enclosing section, keeping the API class/method layout unchanged.

@yeoldegrove yeoldegrove added this to the 2026-11 milestone Nov 11, 2026
@yeoldegrove yeoldegrove self-assigned this Nov 11, 2026
@yeoldegrove

Copy link
Copy Markdown
Contributor Author

@NotTheEvilOne I am planing gldq a document quality checker and I noticed this and it is an "easy fix" ;)

@gardenlinux-docs-bot

Copy link
Copy Markdown

📚 Documentation Preview PR: gardenlinux/docs#241
😎 Deploy Preview: https://deploy-preview-241--gardenlinux-docs.netlify.app

A PR has been created/updated to preview the documentation changes from this PR.

@codecov

codecov Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.56%. Comparing base (e7e6d8a) to head (937885c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #397   +/-   ##
=======================================
  Coverage   91.56%   91.56%           
=======================================
  Files          51       51           
  Lines        3155     3155           
=======================================
  Hits         2889     2889           
  Misses        266      266           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

sphinx-markdown-builder pins rubric and option/signature headings to
level 3 while section titles use their absolute nesting depth. Nested
sphinx-click and sphinxcontrib-autoprogram sections therefore jump from
level 3 to 5 in the generated markdown, which the aggregated docs reject
(gldq HEAD-SKIP).

Override the markdown translator so those headings are rendered one
level below the enclosing section, keeping the API class/method layout
unchanged.

Signed-off-by: Eike Waldt <waldt@b1-systems.de>
On-behalf-of: SAP <eike.waldt@sap.com>
Assisted-by: Kilo:claude-sonnet-4-6
Signed-off-by: Eike Waldt <waldt@b1-systems.de>
On-behalf-of: SAP <eike.waldt@sap.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant