Merge pull request #7 from andybelltree/add-skip-empty-post-option

Move substring after checking for undefined
andybelltree · Aug 29, 2022 · 6ae0eb6 · 6ae0eb6
2 parents 172c98d + b6351d3
commit 6ae0eb6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/dist/main.js b/dist/main.js
@@ -23174,10 +23174,12 @@ async function main$1() {
 
 	const lcov = await parse$2(raw);
 	const baselcov = baseRaw && (await parse$2(baseRaw));
-	const body = diff(lcov, baselcov, options).substring(0, MAX_COMMENT_CHARS);
+	let body = diff(lcov, baselcov, options);
 	if (!body) {
 		console.log(`No changed files in report, exiting...`);
 		return
+	} else {
+		body = body.substring(0, MAX_COMMENT_CHARS);
 	}
 
 	if (shouldDeleteOldComments) {

diff --git a/src/index.js b/src/index.js
@@ -72,10 +72,12 @@ async function main() {
 
 	const lcov = await parse(raw)
 	const baselcov = baseRaw && (await parse(baseRaw))
-	const body = diff(lcov, baselcov, options).substring(0, MAX_COMMENT_CHARS)
+	let body = diff(lcov, baselcov, options)
 	if (!body) {
 		console.log(`No changed files in report, exiting...`)
 		return
+	} else {
+		body = body.substring(0, MAX_COMMENT_CHARS)
 	}
 
 	if (shouldDeleteOldComments) {