fix: only require committer identity in strategy merge

jojomatik · Mar 7, 2022 · e9cbb17 · e9cbb17
1 parent 9f881fc
commit e9cbb17
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -32,10 +32,10 @@ jobs:
           #   Default: `merge`
           strategy: "merge"
           # The name to create merge commits with
-          #   Required
+          #   Required, if strategy `merge` is used
           git_committer_name: ${{ secrets.BOT_GIT_NAME }}
           # The email to create merge commits with
-          #   Required
+          #   Required, if strategy `merge` is used
           git_committer_email: ${{ secrets.BOT_GIT_EMAIL }}
           # The access token to push to the repository
           #   Optional

diff --git a/action.yml b/action.yml
@@ -16,10 +16,10 @@ inputs:
     default: "merge"
   git_committer_name:
     description: 'The name to create merge commits with'
-    required: true
+    required: false
   git_committer_email:
     description: 'The email to create merge commits with'
-    required: true
+    required: false
   github_token:
     description: 'The access token to push to the repository'
     required: false