Merge pull request #36 from gittuf/checkrun-optional

Make checkrun failure return 'neutral' status
gittuf · Dec 12, 2024 · 3442f07 · 3442f07
2 parents f66d69f + 64dfbea
commit 3442f07
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/webhook/webhook.go b/internal/webhook/webhook.go
@@ -458,7 +458,7 @@ func (g *GittufApp) handlePullRequest(ctx context.Context, event *github.PullReq
 				title = "PR is mergeable!"
 				summary = "Sufficient approvals have been submitted for the PR to be mergeable."
 			} else {
-				conclusion = "failure"
+				conclusion = "neutral"
 				title = "PR is not mergeable"
 				summary = "More approvals are necessary for the PR to be mergeable."
 			}
@@ -526,7 +526,7 @@ func (g *GittufApp) handlePullRequest(ctx context.Context, event *github.PullReq
 			title = "PR is mergeable!"
 			summary = "Sufficient approvals have been submitted for the PR to be mergeable."
 		} else {
-			conclusion = "failure"
+			conclusion = "neutral"
 			title = "PR is not mergeable"
 			summary = "More approvals are necessary for the PR to be mergeable."
 		}
@@ -696,7 +696,7 @@ func (g *GittufApp) handlePullRequestReview(ctx context.Context, event *github.P
 		title = "PR is mergeable!"
 		summary = "Sufficient approvals have been submitted for the PR to be mergeable."
 	} else {
-		conclusion = "failure"
+		conclusion = "neutral"
 		title = "PR is not mergeable"
 		summary = "More approvals are necessary for the PR to be mergeable."
 	}