diff --git a/lib/md030.js b/lib/md030.js index aa5c9ba36..9d125fa07 100644 --- a/lib/md030.js +++ b/lib/md030.js @@ -24,24 +24,26 @@ module.exports = { const { line, lineNumber } = item; const match = /^[\s>]*\S+(\s*)/.exec(line); const [ { "length": matchLength }, { "length": actualSpaces } ] = match; - let fixInfo = null; - if ((expectedSpaces !== actualSpaces) && (line.length > matchLength)) { - fixInfo = { - "editColumn": matchLength - actualSpaces + 1, - "deleteCount": actualSpaces, - "insertText": "".padEnd(expectedSpaces) - }; + if (matchLength < line.length) { + let fixInfo = null; + if (expectedSpaces !== actualSpaces) { + fixInfo = { + "editColumn": matchLength - actualSpaces + 1, + "deleteCount": actualSpaces, + "insertText": "".padEnd(expectedSpaces) + }; + } + addErrorDetailIf( + onError, + lineNumber, + expectedSpaces, + actualSpaces, + null, + null, + [ 1, matchLength ], + fixInfo + ); } - addErrorDetailIf( - onError, - lineNumber, - expectedSpaces, - actualSpaces, - null, - null, - [ 1, matchLength ], - fixInfo - ); }); }); } diff --git a/test/bare-list-markers.md b/test/bare-list-markers.md index ebc9403b9..0d9e053aa 100644 --- a/test/bare-list-markers.md +++ b/test/bare-list-markers.md @@ -1,10 +1,33 @@ +# Bare List Markers + +## Ordered 1/1/1 + +1. +1. +1. + +## Ordered 1/2/3 + +1. +2. +3. + +## Unordered * + * +* +* + +## Unordered + ++ ++ + -- +## Unordered - -1. +- +- +- -{MD004:3}{MD004:5} -{MD030:1}{MD030:3}{MD030:5}{MD030:7} + diff --git a/test/detailed-results-MD030-warning-message.md b/test/detailed-results-MD030-warning-message.md index 9f69121cd..add0b90bc 100644 --- a/test/detailed-results-MD030-warning-message.md +++ b/test/detailed-results-MD030-warning-message.md @@ -1,8 +1,16 @@ # - +(bare list item marker) 1. +(bare list item marker) + +-a +(not a list item) + +1.a +(not a list item) - a diff --git a/test/detailed-results-MD030-warning-message.md.fixed b/test/detailed-results-MD030-warning-message.md.fixed index b7b7834b1..f818d37a7 100644 --- a/test/detailed-results-MD030-warning-message.md.fixed +++ b/test/detailed-results-MD030-warning-message.md.fixed @@ -1,8 +1,16 @@ # - +(bare list item marker) 1. +(bare list item marker) + +-a +(not a list item) + +1.a +(not a list item) - a diff --git a/test/detailed-results-MD030-warning-message.results.json b/test/detailed-results-MD030-warning-message.results.json index 1d3ed425a..178ff5972 100644 --- a/test/detailed-results-MD030-warning-message.results.json +++ b/test/detailed-results-MD030-warning-message.results.json @@ -1,24 +1,6 @@ [ { - "lineNumber": 3, - "ruleNames": [ "MD030", "list-marker-space" ], - "ruleDescription": "Spaces after list markers", - "ruleInformation": "https://github.com/DavidAnson/markdownlint/blob/v0.0.0/doc/Rules.md#md030", - "errorDetail": "Expected: 1; Actual: 0", - "errorContext": null, - "errorRange": [1, 1] - }, - { - "lineNumber": 5, - "ruleNames": [ "MD030", "list-marker-space" ], - "ruleDescription": "Spaces after list markers", - "ruleInformation": "https://github.com/DavidAnson/markdownlint/blob/v0.0.0/doc/Rules.md#md030", - "errorDetail": "Expected: 1; Actual: 0", - "errorContext": null, - "errorRange": [1, 2] - }, - { - "lineNumber": 11, + "lineNumber": 19, "ruleNames": [ "MD030", "list-marker-space" ], "ruleDescription": "Spaces after list markers", "ruleInformation": "https://github.com/DavidAnson/markdownlint/blob/v0.0.0/doc/Rules.md#md030", @@ -27,7 +9,7 @@ "errorRange": [1, 3] }, { - "lineNumber": 13, + "lineNumber": 21, "ruleNames": [ "MD030", "list-marker-space" ], "ruleDescription": "Spaces after list markers", "ruleInformation": "https://github.com/DavidAnson/markdownlint/blob/v0.0.0/doc/Rules.md#md030",