From ddd52205b6f87683930f7399dfff8f4c340d4704 Mon Sep 17 00:00:00 2001
From: pvogt09 <50047961+pvogt09@users.noreply.github.com>
Date: Thu, 30 Jul 2020 15:51:23 +0200
Subject: [PATCH] adds signoff option to sign off commits

Signed-off-by: pvogt09 <50047961+pvogt09@users.noreply.github.com>
---
 README.md                      | 1 +
 action.yml                     | 3 +++
 src/create-or-update-branch.ts | 9 +++++++--
 src/create-pull-request.ts     | 4 +++-
 src/main.ts                    | 3 ++-
 5 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index 9a5b5a4..2106f3d 100644
--- a/README.md
+++ b/README.md
@@ -57,6 +57,7 @@ All inputs are **optional**. If not set, sensible defaults will be used.
 | `team-reviewers` | A comma or newline separated list of GitHub teams to request a review from. Note that a `repo` scoped [PAT](https://help.github.com/en/github/authenticating-to-github/creating-a-personal-access-token-for-the-command-line) may be required. See [this issue](https://github.com/peter-evans/create-pull-request/issues/155). | |
 | `milestone` | The number of the milestone to associate this pull request with. | |
 | `draft` | Create a [draft pull request](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests#draft-pull-requests). | `false` |
+| `signoff` | Add Signed-off-by to commit message | `false` |
 
 ### Action outputs
 
diff --git a/action.yml b/action.yml
index 36f51a2..6a65523 100644
--- a/action.yml
+++ b/action.yml
@@ -56,6 +56,9 @@ inputs:
   draft:
     description: 'Create a draft pull request'
     default: false
+  signoff:
+    description: 'Add "Signed-off-by" to commit message'
+    default: false
 outputs:
   pull-request-number:
     description: 'The pull request number'
diff --git a/src/create-or-update-branch.ts b/src/create-or-update-branch.ts
index 9803e69..d927828 100644
--- a/src/create-or-update-branch.ts
+++ b/src/create-or-update-branch.ts
@@ -77,7 +77,8 @@ export async function createOrUpdateBranch(
   commitMessage: string,
   base: string,
   branch: string,
-  branchRemoteName: string
+  branchRemoteName: string,
+  signoff: boolean
 ): Promise<CreateOrUpdateBranchResult> {
   // Get the working base. This may or may not be the actual base.
   const workingBase = await git.symbolicRef('HEAD', ['--short'])
@@ -99,7 +100,11 @@ export async function createOrUpdateBranch(
   if (await git.isDirty(true)) {
     core.info('Uncommitted changes found. Adding a commit.')
     await git.exec(['add', '-A'])
-    await git.commit(['-m', commitMessage])
+    if (signoff == true) {
+      await git.commit(['--signoff', '-m', commitMessage])
+    } else {
+      await git.commit(['-m', commitMessage])
+    }
   }
 
   // Perform fetch and reset the working base
diff --git a/src/create-pull-request.ts b/src/create-pull-request.ts
index 8da5f90..29a0e4a 100644
--- a/src/create-pull-request.ts
+++ b/src/create-pull-request.ts
@@ -23,6 +23,7 @@ export interface Inputs {
   teamReviewers: string[]
   milestone: number
   draft: boolean
+  signoff: boolean
 }
 
 export async function createPullRequest(inputs: Inputs): Promise<void> {
@@ -166,7 +167,8 @@ export async function createPullRequest(inputs: Inputs): Promise<void> {
       inputs.commitMessage,
       inputs.base,
       inputs.branch,
-      branchRemoteName
+      branchRemoteName,
+      inputs.signoff
     )
     core.endGroup()
 
diff --git a/src/main.ts b/src/main.ts
index 3fe5322..1766428 100644
--- a/src/main.ts
+++ b/src/main.ts
@@ -22,7 +22,8 @@ async function run(): Promise<void> {
       reviewers: utils.getInputAsArray('reviewers'),
       teamReviewers: utils.getInputAsArray('team-reviewers'),
       milestone: Number(core.getInput('milestone')),
-      draft: core.getInput('draft') === 'true'
+      draft: core.getInput('draft') === 'true',
+      signoff: core.getInput('signoff') === 'true'
     }
     core.debug(`Inputs: ${inspect(inputs)}`)