-
Notifications
You must be signed in to change notification settings - Fork 41
ci(linting): add remaining linting and formatting checks #301
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(linting): add remaining linting and formatting checks #301
Conversation
670318f to
8ff0c9f
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #301 +/- ##
======================================
Coverage 0.90% 0.90%
======================================
Files 10 10
Lines 551 551
======================================
Hits 5 5
Misses 546 546 🚀 New features to boost your workflow:
|
8ff0c9f to
c3d1b05
Compare
c3d1b05 to
c4bdb49
Compare
tiborsimko
left a comment
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, works nicely 👍
Please add yourself to AUTHORS.md.
c4bdb49 to
e8b55bb
Compare
| @@ -0,0 +1,4 @@ | |||
| # Allow prompt dollar sign in console examples without showing output | |||
| MD014: false | |||
| # Allow multiple headings with same content (for different releases) | |||
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.
Cosmetics: Please separate the two rules for better clarity.
e8b55bb to
4b6dbb7
Compare
Introduce remaining file formatting checks to
run-tests.shand to the GitHub CI:.editorconfigfor consistent character formatting across the repoyamllintfor YAML lintingjsonlintfor JSON lintingshfmtfor shell script formattingmarkdownlintfor markdown lintingprettierfor various other formatting such as.jsfilesCloses #300