Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: allow lobicornis to merge PR where skipped job are present
Browse files Browse the repository at this point in the history
mmatur committed Aug 30, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
1 parent 2fd0ac3 commit 211fcc1
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions pkg/repository/repository_review.go
Original file line number Diff line number Diff line change
@@ -18,6 +18,8 @@ const (
InProgress = "in_progress"
// Queued Check state.
Queued = "queued"
// Skipped Check state.
Skipped = "skipped"

// Approved Review state.
Approved = "APPROVED"
2 changes: 1 addition & 1 deletion pkg/repository/repository_state.go
Original file line number Diff line number Diff line change
@@ -58,7 +58,7 @@ func (r *Repository) getAggregatedState(ctx context.Context, pr *github.PullRequ
return "", err
}

if status == Pending || status == Success {
if status == Pending || status == Success || status == Skipped {
return status, nil
}

0 comments on commit 211fcc1

Please sign in to comment.