-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: TLDR021: Command example should not begin or end in whitespace #310
Conversation
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.
LGTM, thanks for your contribution.
* add tldr-lint dockerfile * Add docker build workflow * npm install -> npm ci * docker push workflow * fix docker publish wf syntax * insert missing tailing newline * tailing newline * improv: update Docker and CI files Signed-off-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com> * cleanup: update CI files Signed-off-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com> * improv: combine publish workflows Signed-off-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com> * ci: add names for CI workflows --------- Signed-off-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com> Co-authored-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com> Co-authored-by: Owen Voke <development@voke.dev>
Signed-off-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com>
Signed-off-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com>
* add tldr 020: spelling of "Note: " * extend tldr020 to command descriptions --------- Co-authored-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com>
@@ -78,7 +78,8 @@ TLDR016 | Label for information link should be spelled exactly `More informa | |||
TLDR017 | Information link should be surrounded with angle brackets | |||
TLDR018 | Page should only include a single information link | |||
TLDR019 | Page should only include a maximum of 8 examples | |||
TLDR020 | Label for additional notes should be spelled exactly `Note: ` | |||
TLDR020 | Label for additional notes should be spelled exactly `Note:` (with a succeeding whitespace) |
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.
I rebased this PR and merged it (sorry VSCode added previous commits too; removed them in the final commit message). To avoid the Markdown lint error, I replaced the space after Note:
with a description. But I have left it as it is in the JS file.
closes #293
This check will now lint cases in which example commands start with a space or end in an unescaped space.