mirror of
https://github.com/DavidAnson/markdownlint.git
synced 2026-04-26 01:36:03 +03:00
[GH-ISSUE #1141] MD050 Regression #623
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#623
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 @mvilrokx on GitHub (Feb 27, 2024).
Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/1141
Problem
When I use
v0.27.0(viamarkdownlint-cli2v0.6.0) the following is raised as an error:When I use
v0.28.1(viamarkdownlint-cli2v0.7.0) the following are raised as errors:When I use
v0.29.0(viamarkdownlint-cli2v0.8.0) or later, no errors are raised.MD050RegressionThe
MD050omission is definitely a regression because it should get reported as a linting error:As you can see, the last strong word (
**kube-state-metrics**) should get flagged.This was caught because our build-platform uses an older version (not under our control) but our GHA is using the latest version.
@DavidAnson commented on GitHub (Feb 27, 2024):
I think you are reporting the following bug where the very first or last word with improper emphasis will not be flagged: https://github.com/DavidAnson/markdownlint/issues/1132
If so, I have already fixed this for the next release. Sorry about that!
@mvilrokx commented on GitHub (Feb 27, 2024):
Possible, I will test when it gets released 👍.
@DavidAnson commented on GitHub (Feb 28, 2024):
It looks like the same thing. You can see that even adding a period at the end is enough to get the second emphasis flagged: https://dlaa.me/markdownlint/#%25m%23%20Issue%201141%0A%0A-%20The%20deployment%20__kube-state-metrics__%20must%20have%20a%20replica%20count%20of%201%20for%20the%20kube-state-metrics.%0A
@DavidAnson commented on GitHub (Feb 28, 2024):
Confirmed duplicate of https://github.com/DavidAnson/markdownlint/issues/1132 and fixed in
nextbranch.