Skip disabling auto-merge on dry run.

reitermarkus · Mar 8, 2023 · f19ed41 · f19ed41
1 parent 94dd605
commit f19ed41
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/automerge-action.ts b/src/automerge-action.ts
@@ -74,15 +74,20 @@ export class AutomergeAction {
     }
   }
 
-  async disableAutoMerge(pullRequest: PullRequest): Promise<DisableAutoMergeMutation> {
+  async disableAutoMerge(pullRequest: PullRequest): Promise<void> {
+    if (this.input.dryRun) {
+      core.info(`Would try disabling auto-merge for pull request ${pullRequest.number}.`)
+      return
+    }
+
     try {
       core.info(`Disabling auto-merge for pull request ${pullRequest.number}.`)
 
       // We need to get the source code of the query since the `@octokit/graphql`
       // API doesn't (yet) support passing a `DocumentNode` object.
       const query = DisableAutoMerge.loc!.source!.body
 
-      return await this.octokit.graphql({
+      await this.octokit.graphql({
         query,
         pullRequestId: pullRequest.node_id,
       })