only output columns if on same line

a-b-r-o-w-n · Jun 2, 2024 · a82d66e · a82d66e
1 parent 8d35d37
commit a82d66e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -173780,12 +173780,14 @@ function processResults(results) {
             else if (severity === 1) {
                 warningCount++;
             }
+            const isMultiLine = endLine && endLine !== line;
             fileAnnotations.push({
                 file: relFilePath,
                 startLine: line,
                 endLine,
-                startColumn: column,
-                endColumn: endColumn,
+                // only add column info if error is on a single line
+                startColumn: isMultiLine ? undefined : column,
+                endColumn: isMultiLine ? undefined : endColumn,
                 severity: severity === 2 ? "failure" : "warning",
                 message: `[${ruleId}] ${message}`,
             });

diff --git a/src/processResults.ts b/src/processResults.ts
@@ -61,12 +61,15 @@ export function processResults(results: ESLint.LintResult[]): LintResult {
         warningCount++;
       }
 
+      const isMultiLine = endLine && endLine !== line;
+
       fileAnnotations.push({
         file: relFilePath,
         startLine: line,
         endLine,
-        startColumn: column,
-        endColumn: endColumn,
+        // only add column info if error is on a single line
+        startColumn: isMultiLine ? undefined : column,
+        endColumn: isMultiLine ? undefined : endColumn,
         severity: severity === 2 ? "failure" : "warning",
         message: `[${ruleId}] ${message}`,
       });