Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds pre-commit hook for markdownlint-cli2 and lints existing code base #195

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

ns-rse
Copy link

@ns-rse ns-rse commented Sep 5, 2024

Closes #194

Adds pre-commit hook markdownlint-cli2 and addresses issues found.

Many were automatically corrected but some required addressing manually. Each linting error that required manually addressing is an individual commit, in a few instances the checks have been disabled and may be addressed subsequently as part of #191.

The .pre-commit-config.yaml includes configuration to run pre-commit.ci but this won't work/run until pre-commit.ci has been authorised to access/work with the repository so someone with that level of access will have to set that up.

Automated fixing of issues identified by [markdownlint-cli2](https://github.com/DavidAnson/markdownlint/).
Adding language to code block starts.
Disabled this check for two `!!!Note` sections which are not code blocks but the lines to include need to be indented.
Disabled this check in two files.
Disabled check to leave links to sections that are yet to be written.
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.

Add pre-commit linking using markdownlint-cli2
2 participants