mirror of
https://github.com/DavidAnson/markdownlint.git
synced 2026-04-25 09:16:02 +03:00
[GH-ISSUE #251] MD037 - Multiple errors of MD037 in a single line is not being fixed at once. #216
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#216
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 @satishdsc on GitHub (Jan 28, 2020).
Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/251
Le **This is test Management ** testing purpose only **Configuration test ** > **test ** > **test d'admin ** > Configuration **test ** test **Configuration test ** > **test ** > **Admin Modèles ** > **Composants test **.@DavidAnson
All MD037 issues are detected in the above statement. I have configured to fix these issues on save. When I save, only 1 issue is fixed. So, I have to hit 'save' 5 times to fix all of them. And on the fifth time, the extension shows all have been fixed. However, some issues are not fixed and not linted as well after hitting save several times. For example, these
**This is test Management ** testing purpose only **Configuration test ** > **test ** > **test d'admin ** >issues are not fixed.@DavidAnson commented on GitHub (Jan 28, 2020):
Is the example above supposed to be one long line? The “>” characters look like they might be used as block quote markers.
@satishdsc commented on GitHub (Jan 30, 2020):
@DavidAnson The example is like small paragraph with multiple lines. This paragraph kind of describes on how to navigate to a particular menu or take same action. So the character ">" is used to show the navigation sequence. For example
To navigate to settings click on **Home** > **my profile** > **settings**. So, I think when we try fix the MD037 issue on this example, space is being removed between ** and > whereas the space between ** and the text should be removed.@DavidAnson commented on GitHub (Jan 31, 2020):
In the first original example, only 2 issues are identified; it takes 5 rounds of fixing to address everything.
In the second original example, I see 2 issues identified; 2 rounds of fixing address everything.
I'll look into this scenario because it seems the library could be doing better at detection - but will caution folks that there will always be scenarios that confound the tool - simply because it's having to deal with behavior that violates the specification and figure out what the user probably meant. Note that for both examples, VS Code's Markdown preview doesn't render anything in bold despite the overwhelming number of "**"'s.
@satishdsc commented on GitHub (Jan 31, 2020):
Yes thank you.