-
Notifications
You must be signed in to change notification settings - Fork 54
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
Fix markdown issues raised by markdownlint #344
base: main
Are you sure you want to change the base?
Conversation
```bash | ||
$ brew install erlang@22 |
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.
Here the dollars vs hashtags are kind of important
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.
Agree, hashtags are comments, revert changes.
```bash | ||
$ mkdir -p ~/.config/rebar3/ && echo "{plugins, [rebar3_hex]}." >> ~/.config/rebar3/rebar.config | ||
mkdir -p ~/.config/rebar3/ && echo "{plugins, [rebar3_hex]}." >> ~/.config/rebar3/rebar.config |
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.
Same here the dollar indicates the command to run and the non dollar indicates the output
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.
RELEASE.md:52:1 MD014/commands-show-output Dollar signs used before commands without showing output [Context: "$ mkdir -p ~/.config/rebar3/ &..."]
|
||
### Booleans | ||
|
||
* avoid `and` and `or` operators, prefer` andalso` and `orelse`, or | ||
* avoid `and` and `or` operators, prefer `andalso` and `orelse`, or |
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 catch here
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.
In general I am against removing the $
and #
from verbatim text. Otherwise it looks like an improvement. Pretty nice tool :)
Ran against this command: markdownlint *.md doc/*.md --disable MD013 MD034 MD010 MD051 MD024 MD029 See a detailed description of the rules is available at https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md
5e73996
to
900b373
Compare
I would keep the If @michalmuskala thinks removing the |
@michalmuskala Any updates on this PR? |
Ran against this command:
See a detailed description of the rules is available at
https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md