feat: add GHPullRequest.markReadyForReview() for draft PRs

Anonycoders · Anonycoders · commit f73fbaea1603 · 2026-01-21T21:02:01.000+01:00
diff --git a/src/main/java/org/kohsuke/github/GHPullRequest.java b/src/main/java/org/kohsuke/github/GHPullRequest.java
@@ -287,10 +287,6 @@ public int getChangedFiles() throws IOException {
         return changedFiles;
     }
 
-    //
-    // details that are only available via get with ID
-    //
-
     /**
      * Gets the closed by.
      *
@@ -539,6 +535,30 @@ public PagedIterable<GHPullRequestReview> listReviews() {
                 .toIterable(GHPullRequestReview[].class, item -> item.wrapUp(this));
     }
 
+    /**
+     * Converts a draft pull request to ready for review.
+     *
+     * @throws IOException
+     *             the io exception
+     * @throws IllegalStateException
+     *             if the pull request is not a draft
+     */
+    public void markReadyForReview() throws IOException {
+        if (!draft) {
+            throw new IllegalStateException("Pull request is not a draft");
+        }
+
+        StringBuilder inputBuilder = new StringBuilder();
+        addParameter(inputBuilder, "pullRequestId", this.getNodeId());
+
+        String graphqlBody = "mutation MarkReadyForReview { markPullRequestReadyForReview(input: {" + inputBuilder
+                + "}) { pullRequest { id } } }";
+
+        root().createGraphQLRequest(graphqlBody).sendGraphQL();
+
+        refresh();
+    }
+
     /**
      * Merge this pull request.
      *
diff --git a/src/test/java/org/kohsuke/github/GHPullRequestTest.java b/src/test/java/org/kohsuke/github/GHPullRequestTest.java
@@ -416,6 +416,25 @@ public void getUserTest() throws IOException {
         }
     }
 
+    /**
+     * Test marking a draft pull request as ready for review.
+     *
+     * @throws Exception
+     *             the exception
+     */
+    @Test
+    public void markReadyForReview() throws Exception {
+        // Create a draft PR first
+        GHPullRequest p = getRepository().createPullRequest("markReadyForReview", "test/mark_ready_for_review", "main", "## test", false, true);
+        assertThat(p.isDraft(), is(true));
+
+        // Mark it ready for review
+        p.markReadyForReview();
+
+        // Verify it's no longer a draft
+        assertThat(p.isDraft(), is(false));
+    }
+
     /**
      * Merge commit SHA.
      *
