-
Notifications
You must be signed in to change notification settings - Fork 792
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
Documentation - allow long lines for tables and code blocks #3049
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3049 +/- ##
==========================================
- Coverage 84.84% 84.75% -0.09%
==========================================
Files 259 259
Lines 9112 9112
==========================================
- Hits 7731 7723 -8
- Misses 1381 1389 +8
|
.github/workflows/markdownlint.yml
Outdated
@@ -2,7 +2,7 @@ name: markdownlint | |||
|
|||
on: | |||
push: | |||
branches: [ main, metrics ] | |||
branches: [ long-lines-linting ] |
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.
intentional? I guess not..
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.
Sure no... I have attached wrong file to the first commit.
It should be in another one, used to execute pipeline on fork.
Fixed afc2d71
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.
Nice improvements! 👍 LGTM.
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.
❤️
Fixes #2917.
Changes
Introduce md lint config file.
Allow long lines for code-block and tables.
Remove obsolete markups.