Skip to content

Adds Markdown Linter #105

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Adds Markdown Linter #105

wants to merge 4 commits into from

Conversation

vpetersson
Copy link
Contributor

  • Adds Markdown linter
    • I used line lenght of 80, which appears to be what @oej has been using
  • Reformatted all Markdown files (which found a few errors too)

@vpetersson vpetersson requested review from oej and madpah as code owners March 13, 2025 16:38
@vpetersson
Adds linter and adds autofixes
024a4fe 
Signed-off-by: Viktor Petersson <[email protected]>
@vpetersson
Change line lenght to 80
b8c525a 
Signed-off-by: Viktor Petersson <[email protected]>
@vpetersson
Adds linting section to readme
3ad334e 
Signed-off-by: Viktor Petersson <[email protected]>
@oej
Copy link
Collaborator

oej commented Mar 14, 2025

Brilliant! I like having a linter and since we have markdown now, I think it's a good idea. I'll take a more detailed look later this weekend.

@madpah
Copy link
Collaborator

madpah commented Apr 17, 2025

@vpetersson - any reason you have chosen not to use an existing GitHub Action for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oej oej Awaiting requested review from oej oej is a code owner

@madpah madpah Awaiting requested review from madpah madpah is a code owner

Assignees

@vpetersson vpetersson

Labels
pr-in-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vpetersson @oej @madpah