mirror of
https://github.com/DavidAnson/markdownlint.git
synced 2026-04-25 17:26:22 +03:00
[GH-ISSUE #234] MD0032: List items without hanging indents should be a violation of this rule #199
Labels
No labels
bug
enhancement
enhancement
enhancement
fixed in next
fixed in next
fixed in next
new rule
new rule
new rule
pull-request
question
refactoring
refactoring
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/markdownlint#199
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @davrodpin on GitHub (Nov 5, 2019).
Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/234
I've been using this plugin on my vscode and mdl as part of my continuous integration checks and I noticed the implementation of MD0032 differs a bit:
List items without hanging indents are a violation of this rule; list items with hanging indents are okay:
Source: https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md#md032---lists-should-be-surrounded-by-blank-lines
@DavidAnson commented on GitHub (Nov 5, 2019):
Someone pointed out a while ago with that hanging indents are a different category than whether or not a list is surrounded by blanks. Therefore, the documentation for the Node.JS implementation of the library does not include the snippet you cite above: https://github.com/DavidAnson/markdownlint/blob/master/doc/Rules.md#md032---lists-should-be-surrounded-by-blank-lines
The pattern above is known as “lazy continuation lines“ and explicitly allowed by the CommonMark specification: https://spec.commonmark.org/0.29/#lazy-continuation-line
This is therefore a deliberate behavioral change in the Node.JS library. If you’d like, it might be interesting to have this discussion in the issue tracker of the MDL tool as maybe they will agree with my logic. Alternatively, for the best consistency between the command-line and VS Code extension, you might consider switching to the Node.JS CLI: https://github.com/igorshubovych/markdownlint-cli
@davrodpin commented on GitHub (Nov 5, 2019):
I will follow your suggestion and try the
markdownlint-clisince the rules implementation is the same and that is exactly what I am looking for.Thank you very much.