chore: when merging always use the pull request title as commitHeadli…

…ne (#34)
Eun · Oct 19, 2023 · c506332 · c506332
1 parent ed6a182
commit c506332
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/pkg/merge-with-label/github/github.go b/pkg/merge-with-label/github/github.go
@@ -292,14 +292,21 @@ func MergePullRequest(
 	token,
 	pullRequestID,
 	expectedHeadOid,
-	mergeStrategy string,
+	mergeStrategy,
+	commitHeadline string,
 ) error {
 	_, err := doGraphQLRequest(ctx, client, token, `
-mutation MergePulLRequest($pullRequestId: ID!, $expectedHeadOid: GitObjectID!, $mergeMethod: PullRequestMergeMethod!){ 
+mutation MergePulLRequest(
+  $pullRequestId: ID!,
+  $expectedHeadOid: GitObjectID!,
+  $mergeMethod: PullRequestMergeMethod!,
+  $commitHeadline: String!
+){ 
   mergePullRequest(input: {
     pullRequestId: $pullRequestId,
     expectedHeadOid: $expectedHeadOid,
     mergeMethod: $mergeMethod,
+    commitHeadline: $commitHeadline,
   }) {
     clientMutationId
   }
@@ -308,6 +315,7 @@ mutation MergePulLRequest($pullRequestId: ID!, $expectedHeadOid: GitObjectID!, $
 		"pullRequestId":   pullRequestID,
 		"expectedHeadOid": expectedHeadOid,
 		"mergeMethod":     mergeStrategy,
+		"commitHeadline":  commitHeadline,
 	})
 	if err != nil {
 		return errors.Wrap(err, "unable to merge pull request")

diff --git a/pkg/merge-with-label/worker/pull_request_worker.go b/pkg/merge-with-label/worker/pull_request_worker.go
@@ -312,6 +312,7 @@ func (worker *pullRequestWorker) mergePullRequest(
 		details.ID,
 		details.LastCommitSha,
 		cfg.Merge.Strategy.GithubString(),
+		details.Title,
 	); err != nil {
 		var graphQLErrors github.GraphQLErrors
 		if errors.As(err, &graphQLErrors) {