Skip to content

Fix nested Markdown list indentation on the website - #276

Merged
larskaare merged 1 commit into
mainfrom
larskaare-markdown-list-indentation
Sep 15, 2026
Merged

larskaare merged 1 commit into
mainfrom
larskaare-markdown-list-indentation

Conversation

@larskaare

Copy link
Copy Markdown
Contributor

Summary

Fixes #239.

MkDocs uses Python-Markdown, which requires four spaces per nested list level. Two-space indentation accepted by GitHub was rendering child bullets as peers on the website.

  • Correct all nested lists in the secret-scanning guideline, including its table of contents and leak-mitigation steps.
  • Fix the same nesting issue in the authentication/authorization and GitHub Actions runner guidelines.
  • Document the four-space indentation convention and website preview command in CONTRIBUTING.md.

Reviewed the other two-space matches; YAML examples and top-level list items were left unchanged. No parser or CSS changes are needed.

Validation

  • uv run --frozen mkdocs build completed successfully using a temporary site output directory.
  • Verified 14 parent-child list relationships in the generated HTML across the three changed pages.
  • git diff --check passed.

The build reported existing missing-anchor links in the GHAS setup and secret-scanning pages; those are outside this indentation fix.

Copilot AI lite review requested due to automatic review settings September 15, 2026 12:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved review comments were identified, and the supplied assessments find it ready for approval.

Pull request overview

Fixes nested Markdown list indentation so child bullets render correctly in the MkDocs website.

Changes:

  • Corrected nested lists across three guideline pages.
  • Documented four-space indentation and local preview guidance.
File summaries
File Description
docs/toolbox/version-control/gh-actions-runners.md Fixed nested action guidance indentation.
docs/toolbox/guidelines/secret-scanning.md Fixed table of contents and nested guidance lists.
docs/toolbox/guidelines/authn-authz.md Fixed nested authentication guidance lists.
CONTRIBUTING.md Added Markdown indentation and preview guidance.
Review details
  • Files reviewed: 1/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

Documentation-only changes were reviewed with no unresolved blocking issues.

Review details
  • Files reviewed: 1/4 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@larskaare
larskaare merged commit 04e19f4 into main Sep 15, 2026
4 checks passed
@larskaare
larskaare deleted the larskaare-markdown-list-indentation branch September 15, 2026 12:28
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.

Bullet point indentation is not preserved in preview site

2 participants