From 0c90d372910e1cb5ecdf06e2179e5b2536aac942 Mon Sep 17 00:00:00 2001 From: Sorena Sarabadani Date: Tue, 27 Jan 2026 01:17:20 +0100 Subject: [PATCH 1/2] fix: override GHPullRequest isPullRequest --- src/main/java/org/kohsuke/github/GHPullRequest.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/src/main/java/org/kohsuke/github/GHPullRequest.java b/src/main/java/org/kohsuke/github/GHPullRequest.java index 6062102c59..40142e5cd6 100644 --- a/src/main/java/org/kohsuke/github/GHPullRequest.java +++ b/src/main/java/org/kohsuke/github/GHPullRequest.java @@ -492,6 +492,16 @@ public boolean isMerged() throws IOException { return merged; } + /** + * Since a GHPullRequest is always a pull request, this method always returns true. + * + * @return true + */ + @Override + public boolean isPullRequest() { + return true; + } + /** * Retrieves all the commits associated to this pull request. * From f82b60ee36305391ad2945164e3dca3ed526de90 Mon Sep 17 00:00:00 2001 From: Sorena Sarabadani Date: Tue, 27 Jan 2026 01:45:53 +0100 Subject: [PATCH 2/2] chore: assert is pull request returning value --- src/test/java/org/kohsuke/github/GHEventPayloadTest.java | 1 + 1 file changed, 1 insertion(+) diff --git a/src/test/java/org/kohsuke/github/GHEventPayloadTest.java b/src/test/java/org/kohsuke/github/GHEventPayloadTest.java index 506bb62269..44e05d3aee 100644 --- a/src/test/java/org/kohsuke/github/GHEventPayloadTest.java +++ b/src/test/java/org/kohsuke/github/GHEventPayloadTest.java @@ -1081,6 +1081,7 @@ public void pull_request() throws Exception { assertThat(event.getPullRequest().getBase().getLabel(), is("baxterthehacker:main")); assertThat(event.getPullRequest().getBase().getSha(), is("9049f1265b7d61be4a8904a9a27120d2064dab3b")); assertThat(event.getPullRequest().isMerged(), is(false)); + assertThat(event.getPullRequest().isPullRequest(), is(true)); assertThat(event.getPullRequest().getMergeable(), nullValue()); assertThat(event.getPullRequest().getMergeableState(), is("unknown")); assertThat(event.getPullRequest().getMergedBy(), nullValue());