Allow disabling commit and push

ageorgou · ageorgou · commit daa593b56978 · 2021-08-22T02:16:49.000+01:00
diff --git a/action.yml b/action.yml
@@ -23,6 +23,9 @@ inputs:
   branch:
     description: "The branch name to push the diagram to (branch will be created if it does not yet exist). For example: diagram"
     required: false
+  push:
+    description: "Whether to push the new commit back to the repository. Default: true"
+    required: false
 runs:
   using: "node12"
   main: "index.js"
diff --git a/src/index.jsx b/src/index.jsx
@@ -67,16 +67,21 @@ const main = async () => {
     return
   }
 
-  await exec('git', ['commit', '-m', commitMessage])
-
-  if (doesBranchExist) {
-    await exec('git', ['push'])
-  } else {
-    await exec('git', ['push', '--set-upstream', 'origin', branch])
-  }
+  const shouldPush = getAsBoolean(core.getInput('push'), true)
+  if (shouldPush) {
+    core.startGroup('Commit and push diagram')
+    await exec('git', ['commit', '-m', commitMessage])
+
+    if (doesBranchExist) {
+      await exec('git', ['push'])
+    } else {
+      await exec('git', ['push', '--set-upstream', 'origin', branch])
+    }
 
-  if (branch) {
-    await exec('git', 'checkout', '-')
+    if (branch) {
+      await exec('git', 'checkout', '-')
+    }
+    core.endGroup()
   }
 
   console.log("All set!")
@@ -106,3 +111,7 @@ function execWithOutput(command, args) {
     }
   })
 }
+
+function getAsBoolean(option, defaultValue) {
+  return option === '' ? defaultValue : (option.toLowerCase() === 'true')
+}
