Merge pull request #216 from timjrobinson/master

Filter out empty strings for major/minor bump checks
phips28 · Jun 11, 2023 · 7ff139d · 7ff139d
2 parents ab01309 + 67ae6a8
commit 7ff139d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -71,8 +71,8 @@ const pkg = getPackageJson();
   }
 
   // input wordings for MAJOR, MINOR, PATCH, PRE-RELEASE
-  const majorWords = process.env['INPUT_MAJOR-WORDING'].split(',');
-  const minorWords = process.env['INPUT_MINOR-WORDING'].split(',');
+  const majorWords = process.env['INPUT_MAJOR-WORDING'].split(',').filter((word) => word != '');
+  const minorWords = process.env['INPUT_MINOR-WORDING'].split(',').filter((word) => word != '');
   // patch is by default empty, and '' would always be true in the includes(''), thats why we handle it separately
   const patchWords = process.env['INPUT_PATCH-WORDING'] ? process.env['INPUT_PATCH-WORDING'].split(',') : null;
   const preReleaseWords = process.env['INPUT_RC-WORDING'] ? process.env['INPUT_RC-WORDING'].split(',') : null;