Merge pull request romeovs#8 from andybelltree/add-skip-empty-post-op…

…tion Remove summary title too if the table is empty.
klausbadelt · Aug 29, 2022 · 53cb1e0 · 53cb1e0
2 parents 6ae0eb6 + 978c69a
commit 53cb1e0
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 20 deletions.
diff --git a/dist/main.js b/dist/main.js
@@ -22986,14 +22986,14 @@ function comment(lcov, options) {
 			: `Coverage for this commit`,
 		table(tbody(tr(th(percentage(lcov).toFixed(2), "%")))),
 		"\n\n",
-		details(
+		reportTable ? details(
 			summary(
 				options.shouldFilterChangedFiles
 					? "Coverage Report for Changed Files"
 					: "Coverage Report",
 			),
 			reportTable,
-		),
+		) : '',
 	)
 }
 
@@ -23029,14 +23029,14 @@ function diff(lcov, before, options) {
 			),
 		),
 		"\n\n",
-		details(
+		reportTable ? details(
 			summary(
 				options.shouldFilterChangedFiles
 					? "Coverage Report for Changed Files"
 					: "Coverage Report",
 			),
 			reportTable,
-		),
+		) : '',
 	)
 }
 

diff --git a/src/comment.js b/src/comment.js
@@ -17,14 +17,16 @@ export function comment(lcov, options) {
 			: `Coverage for this commit`,
 		table(tbody(tr(th(percentage(lcov).toFixed(2), "%")))),
 		"\n\n",
-		details(
-			summary(
-				options.shouldFilterChangedFiles
-					? "Coverage Report for Changed Files"
-					: "Coverage Report",
-			),
-			reportTable,
-		),
+		reportTable
+			? details(
+					summary(
+						options.shouldFilterChangedFiles
+							? "Coverage Report for Changed Files"
+							: "Coverage Report",
+					),
+					reportTable,
+			  )
+			: "",
 	)
 }
 
@@ -60,13 +62,15 @@ export function diff(lcov, before, options) {
 			),
 		),
 		"\n\n",
-		details(
-			summary(
-				options.shouldFilterChangedFiles
-					? "Coverage Report for Changed Files"
-					: "Coverage Report",
-			),
-			reportTable,
-		),
+		reportTable
+			? details(
+					summary(
+						options.shouldFilterChangedFiles
+							? "Coverage Report for Changed Files"
+							: "Coverage Report",
+					),
+					reportTable,
+			  )
+			: "",
 	)
 }