refactor: tweaks output

callstack · Jun 16, 2024 · d6676dd · d6676dd
1 parent e58efac
commit d6676dd
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/packages/compare/src/output/markdown.ts b/packages/compare/src/output/markdown.ts
@@ -208,25 +208,25 @@ function buildRenderIssuesList(issues: RenderIssues | undefined) {
 
   const output = ['Render issues:'];
   if (issues?.initialUpdateCount) {
-    output.push(` - Initial updates: ${formatInitialUpdates(issues.initialUpdateCount)}`);
+    output.push(` - Initial updates: ${formatInitialUpdates(issues.initialUpdateCount, false)}`);
   }
-  if (issues?.redundantUpdates) {
-    output.push(` - Redundant updates: ${formatRedundantUpdates(issues.redundantUpdates)}`);
+  if (issues?.redundantUpdates?.length) {
+    output.push(` - Redundant updates: ${formatRedundantUpdates(issues.redundantUpdates, false)}`);
   }
 
   return output.join('<br/>');
 }
 
-function formatInitialUpdates(count: number | undefined) {
+function formatInitialUpdates(count: number | undefined, showSymbol: boolean = true) {
   if (count == null) return '?';
   if (count === 0) return '-';
 
-  return `${count} 🔴`;
+  return `${count}${showSymbol ? ' 🔴' : ''}`;
 }
 
-function formatRedundantUpdates(redundantUpdates: number[] | undefined) {
+function formatRedundantUpdates(redundantUpdates: number[] | undefined, showSymbol: boolean = true) {
   if (redundantUpdates == null) return '?';
   if (redundantUpdates.length === 0) return '-';
 
-  return `${redundantUpdates.length} (${redundantUpdates.join(', ')}) 🔴`;
+  return `${redundantUpdates.length} (${redundantUpdates.join(', ')})${showSymbol ? ' 🔴' : ''}`;
 }