mirror of
https://github.com/DavidAnson/markdownlint.git
synced 2026-04-25 09:16:02 +03:00
[GH-ISSUE #430] MD053 / Horizontal Rule style complains when horizontal rule in blockquotes #360
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#360
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 @redactedscribe on GitHub (Sep 19, 2021).
Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/430
I'd like to use a horizontal rule inside of blockquotes to help with separation without having to resort to needing to place some text between two separate blockquotes (as suggested in MD028).
Problem is, line 3 complains about consistency due to a plain
---already existing outside of the block quote.I've tried:
But this of course only makes the plain
---to be the one to complain.Is there some way to specify multiple valid
stylevalues? Or maybe the MD rule can be improved to account for this use case (I don't know its implications for all the markdown interpreters)? The "consistent" value should mean the use of the same style of horizontal rule,---vs***, rather than if it's inside a blockquote or not, right?I'm not sure if it's expected, but, if line 2 isn't included as a spacer, line 1 complains with:
and
Is
>a type of header, or is this a bug? I can live with inserting a blank line, but I thought I'd mention this just in case.Thanks.
markdownlint v0.44.0
@DavidAnson commented on GitHub (Sep 19, 2021):
The block quoted HR should not be reported as a consistency violation. Here's a reproduction: https://dlaa.me/markdownlint/#%25m%23%20Issue%20430%0A%0A---%0A%0A%3E%20Text%0A%3E%0A%3E%20---%0A%3E%20Text%0A
@DavidAnson commented on GitHub (Sep 19, 2021):
The messages when blank line L2 are not present are correct; that syntax is interpreted as a setext heading (note bold L1 text): https://dlaa.me/markdownlint/#%25m%23%20Issue%20430%0A%0A%3E%20Text%0A%3E%20---%0A%3E%20Text%0A
@redactedscribe commented on GitHub (Dec 28, 2021):
Hello @DavidAnson, thanks for the fix.
I was wondering if this will be applied to vscode-markdownlint as well? At the time of posting, I wasn't aware that there was also a vscode repo.
@DavidAnson commented on GitHub (Dec 28, 2021):
Yes. Next update will be to markdownlint-cli2, then the VS Code extension.
@redactedscribe commented on GitHub (Dec 28, 2021):
Great news! And by the looks of it, you're closing in on being able to fix the presence of parentheses in paths issue. Looking forward to being able to make use of this extension again (most my *.md files reside within such a path).