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

docs: add markdown linter #1270

Merged
merged 2 commits into from
Feb 10, 2025
Merged

docs: add markdown linter #1270

merged 2 commits into from
Feb 10, 2025

Conversation

drahnr
Copy link
Contributor

@drahnr drahnr commented Feb 7, 2025

All our documentation is in markdown format, but it's not linted for consistency. This PR adds markdownlint-cli2 checks per PR, to replicated locally a Makefile rule was added.


This change is Reviewable

@karlem karlem force-pushed the bernhard-rename-infra-to-deployment branch from b5a2e8c to 8d30fbe Compare February 7, 2025 11:19
@drahnr drahnr force-pushed the bernhard-add-markdown-linter branch 2 times, most recently from debbe59 to 6c4ec16 Compare February 7, 2025 13:19
@drahnr drahnr marked this pull request as ready for review February 7, 2025 19:34
@drahnr drahnr requested a review from a team as a code owner February 7, 2025 19:34
@drahnr drahnr changed the title doc: add markdown linter docs: add markdown linter Feb 10, 2025
@drahnr drahnr force-pushed the bernhard-rename-infra-to-deployment branch 2 times, most recently from 3c1cf4f to 076d4ed Compare February 10, 2025 08:31
@drahnr drahnr changed the base branch from bernhard-rename-infra-to-deployment to main February 10, 2025 12:52
@drahnr drahnr force-pushed the bernhard-add-markdown-linter branch from ca864d4 to 174b6e3 Compare February 10, 2025 13:24
@drahnr drahnr merged commit ef29728 into main Feb 10, 2025
1 check passed
@drahnr drahnr deleted the bernhard-add-markdown-linter branch February 10, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants