-
Notifications
You must be signed in to change notification settings - Fork 838
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
chore: add markdown lint check to default lint command #5074
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5074 +/- ##
==========================================
+ Coverage 93.26% 93.27% +0.01%
==========================================
Files 317 317
Lines 8195 8195
Branches 1641 1641
==========================================
+ Hits 7643 7644 +1
+ Misses 552 551 -1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for taking the time to update this and the CONTRIBUTING.md
.
Please also adapt the lint workflow - so that the script does not run twice in CI 🙂
Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
done |
…y#5074) Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
Add the lint checks for markdown to the npm run lint command, to align the local command with what is being executed on github workflow.
Without this change the lint was taking 31s, and after it kept at 31s.