Skip to content

Fix Markdown lint warnings and lint all files - #220

Merged
orangecms merged 8 commits into
mainfrom
philip/markdown-lint
Nov 9, 2026
Merged

orangecms merged 8 commits into
mainfrom
philip/markdown-lint

Conversation

@pamolloy

@pamolloy pamolloy commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

The Markdown Lint workflow has been failing on main since 9b366bc added a code block with no language (MD040) in components.md. This PR fixes that warning and the remaining warnings in the files on the lint skip list. It then replaces lazy-lint.bash with a plain mdl ., and git_recurse in .mdlrc keeps the generated book/ directory from being linted.

- MD041 First line in file should be a top level header

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
- MD036 Emphasis used instead of a header
- MD040 Fenced code blocks should have a language specified

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
- MD040 Fenced code blocks should have a language specified

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
- MD040 Fenced code blocks should have a language specified

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
- MD040 Fenced code blocks should have a language specified

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
- MD025 Multiple top level headers in the same document
- MD040 Fenced code blocks should have a language specified

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
Skip the generated book/ directory, which is ignored by git and removed
by `mdbook clean`, when linting with `mdl .`

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
All files on the list of shame now pass mdl, so drop lazy-lint.bash and
run mdl on the whole repository

Signed-off-by: Philip Molloy <philip@philipmolloy.com>
@pamolloy
pamolloy requested a review from orangecms November 9, 2026 19:21
@orangecms
orangecms merged commit 3b355e5 into main Nov 9, 2026
3 checks passed
@orangecms
orangecms deleted the philip/markdown-lint branch November 9, 2026 19:24
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.

2 participants