Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Github workflows - Markdown lint tool #537

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Github workflows - Markdown lint tool #537

wants to merge 1 commit into from

Conversation

Prome88
Copy link
Contributor

@Prome88 Prome88 commented Apr 29, 2020

Github workflows - Markdown lint tool for documentation.

The lint will fail due to few issues in the documentation MD files. Should I also fix those reported issues or wait for v9.0 documentation to do so?

@MathewSachin
Copy link
Owner

I tried looking around a bit if we can have the lint output viewable without having to see the whole GitHub action logs. It won't be very helpful otherwise. It should use GitHub Actions annotations and show like in this PR: ember-template-lint/ember-template-lint#966

And, do fix the issues for the current files otherwise this will just display all pull-requests as failing to pass tests.

Github workflows - Markdown lint tool for documentation
@Prome88
Copy link
Contributor Author

Prome88 commented Jun 16, 2020

Documentation has been updated to reflect the markdown linter. Implementing viewable output is unfortunately beyond my capabilities.

Should I remove the linter from the commit and just push the updated documentation?

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