fix: don't fail versioning because auto-merge failed to enable (#50)

ExodusMovement · Jun 28, 2024 · d71ceb4 · d71ceb4
1 parent fba4c3a
commit d71ceb4
Show file tree

Hide file tree

Showing 5 changed files with 75 additions and 41 deletions.
diff --git a/dist/publish/index.js b/dist/publish/index.js
diff --git a/dist/publish/index.js.map b/dist/publish/index.js.map
diff --git a/dist/version/index.js b/dist/version/index.js
diff --git a/dist/version/index.js.map b/dist/version/index.js.map
diff --git a/src/utils/github.ts b/src/utils/github.ts
@@ -2,6 +2,7 @@ import * as core from '@actions/core'
 import * as github from '@actions/github'
 
 import { Repo } from './types'
+import { unwrapErrorMessage } from './errors'
 
 export type GithubClient = ReturnType<typeof github.getOctokit>
 
@@ -85,29 +86,35 @@ export async function createPullRequest({
   }
 
   if (autoMerge) {
-    const autoMergePromise = client.graphql<EnablePullRequestAutoMergeResponse>(
-      /* GraphQL */ `
-        mutation EnableAutoMerge($pullRequestId: ID!) {
-          enablePullRequestAutoMerge(
-            input: { pullRequestId: $pullRequestId, mergeMethod: SQUASH }
-          ) {
-            pullRequest {
-              autoMergeRequest {
-                enabledAt
+    const autoMergePromise = client
+      .graphql<EnablePullRequestAutoMergeResponse>(
+        /* GraphQL */ `
+          mutation EnableAutoMerge($pullRequestId: ID!) {
+            enablePullRequestAutoMerge(
+              input: { pullRequestId: $pullRequestId, mergeMethod: SQUASH }
+            ) {
+              pullRequest {
+                autoMergeRequest {
+                  enabledAt
+                }
               }
             }
           }
+        `,
+        {
+          pullRequestId: response.data.node_id,
         }
-      `,
-      {
-        pullRequestId: response.data.node_id,
-      }
-    )
+      )
+      .then(({ enablePullRequestAutoMerge: { pullRequest } }) =>
+        core.debug(`Auto-merge enabled at ${pullRequest.autoMergeRequest.enabledAt}`)
+      )
+      .catch((error) => {
+        core.warning(
+          `Failed to enable auto-merge: ${unwrapErrorMessage(error, 'for unknown reasons')}`
+        )
+      })
 
     promises.push(autoMergePromise)
-    autoMergePromise.then(({ enablePullRequestAutoMerge: { pullRequest } }) =>
-      core.debug(`Auto-merge enabled at ${pullRequest.autoMergeRequest.enabledAt}`)
-    )
   }
 
   await Promise.all(promises)