Apply suggestions from review

Author: ageorgou

README.md

@@ -52,13 +52,13 @@ The maximum number of nested folders to show files within. A higher number will
 
 Default: 9
 
-### `push`
+### `should_push`
 
 Whether to make a new commit with the diagram and push it to the original repository.
 
 Should be a boolean value, i.e. `true` or `false`. See `commit_message` and `branch` for how to customise the commit.
 
-Default: `false`
+Default: `true`
 
 ### `commit_message`
 
@@ -126,7 +126,7 @@ Example:
     path: downloads
   # Diagram now available at downloads/images/diagram.svg
 ```
-Note that this will still also create a commit, unless you specify `push: false`!
+Note that this will still also create a commit, unless you specify `should_push: false`!
 
 Alternatively, the SVG description of the diagram is available in the `svg` output,
 which you can refer to in your workflow as e.g. `${{ steps.make_diagram.outputs.svg }}`.

action.yml

@@ -23,7 +23,7 @@ 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:
+  should_push:
     description: "Whether to push the new commit back to the repository. Must be true or false. Default: true"
     required: false
     default: true

index.js

@@ -26299,7 +26299,7 @@ var main = async () => {
     core.info("[INFO] No changes to the repo detected, exiting");
     return;
   }
-  const shouldPush = core.getBooleanInput("push");
+  const shouldPush = core.getBooleanInput("should_push");
   if (shouldPush) {
     core.startGroup("Commit and push diagram");
     await (0, import_exec.exec)("git", ["commit", "-m", commitMessage]);

src/index.jsx

@@ -70,7 +70,7 @@ const main = async () => {
     return
   }
 
-  const shouldPush = core.getBooleanInput('push')
+  const shouldPush = core.getBooleanInput('should_push')
   if (shouldPush) {
     core.startGroup('Commit and push diagram')
     await exec('git', ['commit', '-m', commitMessage])