Skip to content

Add Markdown checks#516

Merged
frankieroberto merged 1 commit into
mainfrom
add-markdown-checks
Jun 12, 2026
Merged

Add Markdown checks#516
frankieroberto merged 1 commit into
mainfrom
add-markdown-checks

Conversation

@frankieroberto

Copy link
Copy Markdown
Collaborator

This adds a script which will do some basic Markdown syntax checks (currently only checking for the redundant presence of a redundant h1).

It can be run locally, or will run automatically on any open pull requests, with the result being a comment left (or deleted once resolved).

Copied from https://github.com/NHSDigital/digital-prevention-services-portfolio/tree/main/.github where we've been testing it out.

This adds a script which will do some basic Markdown syntax checks (currently only checking for the redundant presence of a redundant h1).

It can be run locally, or will run automatically on any open pull requests, with the result being a comment left (or deleted once resolved).

Copied from https://github.com/NHSDigital/digital-prevention-services-portfolio/tree/main/.github where we've been testing it out.
@edwardhorsford edwardhorsford temporarily deployed to dsps-design-add-markdow-qjjaoq June 1, 2026 15:48 Inactive
@frankieroberto

Copy link
Copy Markdown
Collaborator Author

This would have caught #514 (comment) - so feels like a useful thing to copy across here?

We can extend it in future to cover other potential common Markdown issues?

@edwardhorsford edwardhorsford left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tried this locally with the help of Claude, seemed to work.

It suggested that if a fenced code block contained an h1 then this would flag. This could ignore on any fenced code blocks - but I think in practice this is likely a very unlikely scenario for the design history.

@frankieroberto

Copy link
Copy Markdown
Collaborator Author

@edwardhorsford ah good point Claude. But probably unlikely to come up. We can iterate later if needed.

Ideas and pull requests on other checks we can make welcome!

@frankieroberto frankieroberto merged commit 0f2b815 into main Jun 12, 2026
4 checks passed
@frankieroberto frankieroberto deleted the add-markdown-checks branch June 12, 2026 13:33
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