Skip to content

fix: remark-lint unordered-list-marker-style rule#2680

Merged
XhmikosR merged 5 commits intonodejs:masterfrom
nschonni:fix--remark-lint-unordered-list-marker-style-rule
Oct 16, 2019
Merged

fix: remark-lint unordered-list-marker-style rule#2680
XhmikosR merged 5 commits intonodejs:masterfrom
nschonni:fix--remark-lint-unordered-list-marker-style-rule

Conversation

@nschonni
Copy link
Copy Markdown
Member

Switch to asterisks style to match upstream

Comment thread locale/ar/about/releases.md Outdated
@nschonni nschonni force-pushed the fix--remark-lint-unordered-list-marker-style-rule branch from c4f44a0 to abe3d29 Compare October 14, 2019 06:32
Comment thread locale/en/knowledge/HTTP/servers/how-to-serve-static-files.md Outdated
Comment thread locale/en/blog/vulnerability/v8-memory-corruption-stack-overflow.md Outdated
Comment thread locale/en/get-involved/node-meetups.md Outdated
@nschonni nschonni force-pushed the fix--remark-lint-unordered-list-marker-style-rule branch from abe3d29 to 783c8bc Compare October 14, 2019 06:48
@XhmikosR
Copy link
Copy Markdown
Contributor

Also, please add the markdownlint rule; even if we replace markdownlint with remark-lint, this should be there. https://github.com/DavidAnson/markdownlint/blob/master/doc/Rules.md#md004---unordered-list-style

Switch to asterisks style to match upstream
@nschonni nschonni force-pushed the fix--remark-lint-unordered-list-marker-style-rule branch from 783c8bc to 1e9e364 Compare October 14, 2019 07:06
Comment thread static/legacy/README.md Outdated
Comment thread static/legacy/README.md
Comment thread static/legacy/README.md
@XhmikosR
Copy link
Copy Markdown
Contributor

I already reverted those @Trott

@XhmikosR XhmikosR merged commit 221e35b into nodejs:master Oct 16, 2019
@nschonni nschonni deleted the fix--remark-lint-unordered-list-marker-style-rule branch October 16, 2019 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants