Skip to content

Add markdownlint to ensure consistent formatting #14

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

Merged
merged 5 commits into from
Mar 28, 2025
Merged

Conversation

michaelin
Copy link
Contributor

I've seen too many places where markdown becomes hard to read over time due to different peoples preferences wrt. formatting and style. Starting out with a unified formatting strategy can lessen this problem.

I've added a GitHub Action for validating all markdown in the repo and a small .markdownlint.yaml config file with all defaults except for disabling line length requirements. We can discuss further changes from there.

Verified

This commit was signed with the committer’s verified signature.
michaelin Michael Ingeman-Nielsen
A new linting job has been added to the PR workflow,
and both workflows now triggers on changes to themselves as well
as all markdown files in the root directory and Hugo dirs.

Verified

This commit was signed with the committer’s verified signature.
michaelin Michael Ingeman-Nielsen
All default MarkdownLint rules are enabled except for MD013/line-length. This rule is often debated, and its value is dependent on your editors word wrap features and configuration.

Verified

This commit was signed with the committer’s verified signature.
michaelin Michael Ingeman-Nielsen
@michaelin michaelin marked this pull request as ready for review February 21, 2025 13:58
Copy link
Contributor

@housa housa left a comment

Choose a reason for hiding this comment

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

Awesome idea. Sorry for the late reply, but totally onboard with this. <3

moller2866 and others added 2 commits March 28, 2025 13:42

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@moller2866 moller2866 merged commit 75ff44f into main Mar 28, 2025
2 checks passed
@moller2866 moller2866 deleted the markdownlint branch March 28, 2025 13:04
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.

None yet

3 participants