Skip to content
This repository was archived by the owner on Oct 4, 2022. It is now read-only.

chore: add markdownlint linter to docs + fix current one#141

Merged
sipsma merged 2 commits into
dagger:mainfrom
grouville:markdownlint
Aug 31, 2022
Merged

chore: add markdownlint linter to docs + fix current one#141
sipsma merged 2 commits into
dagger:mainfrom
grouville:markdownlint

Conversation

@grouville
Copy link
Copy Markdown
Member

  • Add markdownlint linter until we doogfood it with Dagger. It follows Dagger's current policy on markdown
  • Goes in same direction as PR docs: Add docs website #103 (conjointly) @jpadams. It is complementary to this PR

Add markdownlint to keep docs consistency

Signed-off-by: Guillaume de Rouville <guillaume.derouville@gmail.com>
Fix current docs to please markdownlint. It is follows the same usage as our current docs

Signed-off-by: Guillaume de Rouville <guillaume.derouville@gmail.com>
@grouville grouville requested a review from jpadams August 30, 2022 17:25
Copy link
Copy Markdown
Contributor

@sipsma sipsma left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@sipsma sipsma merged commit 1e7d379 into dagger:main Aug 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants