Fixing the issue when status check per rule matches multiple actions. (#29628)

charles7668 · charles7668 · commit 0c0ea5d57e6f · 2024-03-06T20:14:24.000+08:00
diff --git a/services/pull/commit_status.go b/services/pull/commit_status.go
@@ -51,10 +51,6 @@ func MergeRequiredContextsCommitStatus(commitStatuses []*git_model.CommitStatus,
 		}
 	}
 
-	if matchedCount != len(requiredContexts) {
-		return structs.CommitStatusPending
-	}
-
 	if matchedCount == 0 {
 		status := git_model.CalcCommitStatus(commitStatuses)
 		if status != nil {

Original file line number	Diff line number	Diff line change
`@@ -51,10 +51,6 @@ func MergeRequiredContextsCommitStatus(commitStatuses []*git_model.CommitStatus,`
`51`	`51`	`}`
`52`	`52`	`}`
`53`	`53`
`54`		`- if matchedCount != len(requiredContexts) {`
`55`		`- return structs.CommitStatusPending`
`56`		`- }`
`57`		`-`
`58`	`54`	`if matchedCount == 0 {`
`59`	`55`	`status := git_model.CalcCommitStatus(commitStatuses)`
`60`	`56`	`if status != nil {`