docs(audit): change warning sign in reports (#126)

graphql · Aug 27, 2024 · a1cc086 · a1cc086
1 parent d1ee3de
commit a1cc086
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/audits/render.ts b/src/audits/render.ts
@@ -39,8 +39,7 @@ export async function renderAuditResultsToHTML(results: AuditResult[]) {
     report += `<li><span style="font-family: monospace">💡</span> <b>${grouped.notice.length}</b> notices (suggestions)</li>\n`;
   }
   if (grouped.warn.length) {
-    // TODO: warning sign is rendered as "⚠️" in markdown instead of the emoji
-    report += `<li><span style="font-family: monospace">⚠️</span> <b>${grouped.warn.length}</b> warnings (optional)</li>\n`;
+    report += `<li><span style="font-family: monospace">❗️</span> <b>${grouped.warn.length}</b> warnings (optional)</li>\n`;
   }
   if (grouped.error.length) {
     report += `<li><span style="font-family: monospace">❌</span> <b>${grouped.error.length}</b> errors (required)</li>\n`;

diff --git a/tests/audits.test.ts b/tests/audits.test.ts
@@ -119,7 +119,7 @@ describe('Render audit results to HTML', () => {
       <ul>
       <li><b>6</b> audits in total</li>
       <li><span style="font-family: monospace">✅</span> <b>2</b> pass</li>
-      <li><span style="font-family: monospace">⚠️</span> <b>2</b> warnings (optional)</li>
+      <li><span style="font-family: monospace">❗️</span> <b>2</b> warnings (optional)</li>
       <li><span style="font-family: monospace">❌</span> <b>2</b> errors (required)</li>
       </ul>