Skip to content

Commit f4462c6

Browse files
committed
Attempt to fix polynomial regular expression warning in formatter-template by making the pattern more explicit.
1 parent 358d2dc commit f4462c6

5 files changed

+11
-11
lines changed

formatter-template/markdownlint-cli2-formatter-template.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
"use strict";
44

55
// eslint-disable-next-line prefer-named-capture-group
6-
const tokenRe = /\$\{([^:!}]+)(?:([:!])([^{}]*\{[^{}]+\}[^{}]*|[^}]+))?\}/igu;
6+
const tokenRe = /\$\{(fileName|lineNumber|columnNumber|ruleName|ruleDescription|ruleInformation|errorContext|errorDetail)(?:([:!])([^{}]*\{[^{}]+\}[^{}]*|[^}]+))?\}/igu;
77

88
// Output markdownlint-cli2 results using a template
99
const outputFormatter = (options, params) => {

test/snapshots/markdownlint-cli2-test-exec.js.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -5139,11 +5139,11 @@ Generated by [AVA](https://avajs.dev).
51395139
##vso[task.logissue type=error;sourcepath=viewme.md;linenumber=6;code=MD025/single-title/single-h1]Multiple top-level headings in the same document␊
51405140
##vso[task.logissue type=error;sourcepath=viewme.md;linenumber=12;columnumber=4;code=MD019/no-multiple-space-atx]Multiple spaces after hash on atx style heading␊
51415141
##vso[task.logissue type=error;sourcepath=viewme.md;linenumber=14;columnumber=14;code=MD047/single-trailing-newline]Files should end with a single newline character␊
5142-
#Undefined=#Column=10#Column=10##Column=10#␊
5143-
#Undefined=#Column=##No column number#No column number#␊
5144-
#Undefined=#Column=##No column number#No column number#␊
5145-
#Undefined=#Column=4#Column=4##Column=4#␊
5146-
#Undefined=#Column=14#Column=14##Column=14#`,
5142+
#Undefined=${undefined}#Column=10#Column=10##Column=10#␊
5143+
#Undefined=${undefined}#Column=##No column number#No column number#␊
5144+
#Undefined=${undefined}#Column=##No column number#No column number#␊
5145+
#Undefined=${undefined}#Column=4#Column=4##Column=4#␊
5146+
#Undefined=${undefined}#Column=14#Column=14##Column=14#`,
51475147
stdout: `markdownlint-cli2 vX.Y.Z (markdownlint vX.Y.Z)␊
51485148
Finding: **/*.md␊
51495149
Linting: 1 file(s)␊
Binary file not shown.

test/snapshots/markdownlint-cli2-test-main.js.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -4377,11 +4377,11 @@ Generated by [AVA](https://avajs.dev).
43774377
##vso[task.logissue type=error;sourcepath=viewme.md;linenumber=6;code=MD025/single-title/single-h1]Multiple top-level headings in the same document␊
43784378
##vso[task.logissue type=error;sourcepath=viewme.md;linenumber=12;columnumber=4;code=MD019/no-multiple-space-atx]Multiple spaces after hash on atx style heading␊
43794379
##vso[task.logissue type=error;sourcepath=viewme.md;linenumber=14;columnumber=14;code=MD047/single-trailing-newline]Files should end with a single newline character␊
4380-
#Undefined=#Column=10#Column=10##Column=10#␊
4381-
#Undefined=#Column=##No column number#No column number#␊
4382-
#Undefined=#Column=##No column number#No column number#␊
4383-
#Undefined=#Column=4#Column=4##Column=4#␊
4384-
#Undefined=#Column=14#Column=14##Column=14#␊
4380+
#Undefined=${undefined}#Column=10#Column=10##Column=10#␊
4381+
#Undefined=${undefined}#Column=##No column number#No column number#␊
4382+
#Undefined=${undefined}#Column=##No column number#No column number#␊
4383+
#Undefined=${undefined}#Column=4#Column=4##Column=4#␊
4384+
#Undefined=${undefined}#Column=14#Column=14##Column=14#␊
43854385
`,
43864386
stdout: `markdownlint-cli2 vX.Y.Z (markdownlint vX.Y.Z)␊
43874387
Finding: **/*.md␊
Binary file not shown.

0 commit comments

Comments
 (0)