-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add CI linting for Markdown files #16
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - good idea to add linting checks
40cf273
to
f0e7c8d
Compare
I've made some fairly substantial changes to how this works (moving from an Action to a Docker image) so I can use the custom one-sentence-per-line rule - requesting re-review as a result |
f0e7c8d
to
5ba643e
Compare
Add ADR folder structure and templates
9086d6b
to
39f367f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes to linting LGTM
Adds linting using markdown lint for ADRs and the README
bbc1236
to
48a9f03
Compare
Details
Adds linting of Markdown files in the repository using https://github.com/DavidAnson/markdownlint-cli2
Might make it a bit easier to keep the docs neat (and it found a misnumbering for me!). Thoughts?
Pivotal Story
Story URL: Done as part of https://www.pivotaltracker.com/story/show/186644493
Related PRs
Follows #15
Links to external test runs/working deployment
N/A
Submitter PR Checks
(tick as appropriate)
Reviewer PR Checks
(tick as appropriate)
Info on Cloudfit PRs