add red modifiers

dequelabs · Feb 13, 2025 · 5d298cb · 5d298cb
1 parent 9db43a0
commit 5d298cb
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 13 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -43079,17 +43079,15 @@ ${pendingInterceptorsFormatter.format(pending)}
               // Add file heading
               core.summary
                 .addHeading(
-                  `Error${(0, utils_1.pluralize)(errors.length)} in ${file}:`,
+                  `❌ Error${(0, utils_1.pluralize)(errors.length)} in ${file}:`,
                   2
                 )
-                .addRaw('<span style="color: red">', true)
                 .addList(
                   errors.map(
                     (error) =>
-                      `Line ${error.line}: ${error.ruleId} - ${error.description}`
+                      `🔴 Line ${error.line}: ${error.ruleId} - ${error.description}`
                   )
                 )
-                .addRaw('</span>', true)
                 .addBreak()
               // Create GitHub annotations
               for (const error of errors) {

diff --git a/src/linter.test.ts b/src/linter.test.ts
@@ -452,21 +452,22 @@ describe('linter', () => {
 
       // Verify file sections
       assert.isTrue(
-        summaryStub.addHeading.calledWith('Error in app.js:', 3),
+        summaryStub.addHeading.calledWith('❌ Error in app.js:', 2),
         'Should add app.js section'
       )
+
       assert.isTrue(
-        summaryStub.addHeading.calledWith('Errors in index.html:', 3),
+        summaryStub.addHeading.calledWith('❌ Errors in index.html:', 2),
         'Should add index.html section'
       )
 
       // Verify error lists
       const appJsErrors = [
-        'Line 1: click-handler - Click handler should have keyboard equivalent'
+        '🔴 Line 1: click-handler - Click handler should have keyboard equivalent'
       ]
       const indexHtmlErrors = [
-        'Line 1: color-contrast - Element has insufficient color contrast',
-        'Line 1: aria-label - Element should have aria-label'
+        '🔴 Line 1: color-contrast - Element has insufficient color contrast',
+        '🔴 Line 1: aria-label - Element should have aria-label'
       ]
 
       assert.isTrue(

diff --git a/src/linter.ts b/src/linter.ts
@@ -72,15 +72,13 @@ export async function lintFiles(
     if (errors.length > 0) {
       // Add file heading
       core.summary
-        .addHeading(`Error${pluralize(errors.length)} in ${file}:`, 2)
-        .addRaw('<span style="color: red">', true)
+        .addHeading(`❌ Error${pluralize(errors.length)} in ${file}:`, 2)
         .addList(
           errors.map(
             (error) =>
-              `Line ${error.line}: ${error.ruleId} - ${error.description}`
+              `🔴 Line ${error.line}: ${error.ruleId} - ${error.description}`
           )
         )
-        .addRaw('</span>', true)
         .addBreak()
 
       // Create GitHub annotations