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

ci: add check-docs workflow #93

Merged
merged 16 commits into from
May 20, 2023
Merged

ci: add check-docs workflow #93

merged 16 commits into from
May 20, 2023

Conversation

piyoki
Copy link
Contributor

@piyoki piyoki commented May 17, 2023

Background

Apply markdown linting cuz most of the time, as human beings, we may make typos very easily.

Preview: https://github.com/yqlbu/dae/tree/ci-markdown-lint

Checklist

  • The Pull Request has been fully tested

Full changelog

  • ci: add check-docs workflow
  • ci: ONLY kick off build when targeting workflow file changes
  • fix: apply syntax fix for all problematic .md files

Issue reference

NA.

@piyoki piyoki added the ci label May 17, 2023
@piyoki piyoki requested a review from mzz2017 as a code owner May 17, 2023 15:34
@piyoki piyoki self-assigned this May 17, 2023
@piyoki piyoki requested a review from a team as a code owner May 17, 2023 15:43
@piyoki piyoki marked this pull request as draft May 17, 2023 15:46
@piyoki piyoki marked this pull request as ready for review May 17, 2023 16:17
@piyoki piyoki requested a review from mzz2017 May 20, 2023 16:24
@mzz2017
Copy link
Contributor

mzz2017 commented May 20, 2023

Good job!

@mzz2017 mzz2017 merged commit eac04e9 into daeuniverse:main May 20, 2023
@piyoki piyoki deleted the ci-markdown-lint branch May 20, 2023 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants