fix: handling of boolean input

all inputs are strings even though their yaml definition suggests that they may of other types too. see actions/toolkit#361
fastify · Mar 4, 2021 · afbe82b · afbe82b
1 parent 8afc7f5
commit afbe82b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "github-action-merge-dependabot",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "",
   "main": "src/index.js",
   "scripts": {

diff --git a/src/util.js b/src/util.js
@@ -5,14 +5,16 @@ const { logWarning } = require('./log')
 const mergeMethods = {
   merge: 'merge',
   squash: 'squash',
-  rebase: 'rebase'
+  rebase: 'rebase',
 }
 
 const getMergeMethod = () => {
   const input = core.getInput('merge-method')
 
   if (!input || !mergeMethods[input]) {
-    logWarning('merge-method input is ignored because it is malformed, defaulting to `squash`.')
+    logWarning(
+      'merge-method input is ignored because it is malformed, defaulting to `squash`.'
+    )
     return mergeMethods.squash
   }
 
@@ -24,5 +26,5 @@ exports.getInputs = () => ({
   MERGE_METHOD: getMergeMethod(),
   EXCLUDE_PKGS: core.getInput('exclude') || [],
   MERGE_COMMENT: core.getInput('merge-comment') || '',
-  APPROVE_ONLY: core.getInput('approve-only')
+  APPROVE_ONLY: /true/i.test(core.getInput('approve-only')),
 })