github/drone
5
0
Fork 0
mirror of https://github.com/harness/drone.git synced 2025-05-04 23:22:42 +08:00
Code Issues Packages Projects Releases Wiki Activity

remove branch rule: all status checks must succeed (#691)

Browse Source
This commit is contained in:
Marko Gacesa 2023-10-18 16:33:55 +00:00 committed by Harness
parent d2b971c49a
commit c0849b7b36
1 changed files with 1 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
app/services/protection/rule_branch.go
View File

@ -103,16 +103,6 @@ func (v *Branch) CanMerge(_ context.Context, in CanMergeInput) (CanMergeOutput,
}
}
if v.PullReq.StatusChecks.AllMustSucceed {
for i := range in.Checks {
if in.Checks[i].Status != enum.CheckStatusSuccess {
violations.Add("pullreq.status_checks.all_must_succeed",
"All status checks must complete successfully.")
break
}
}
}
// pullreq.merge
if len(v.PullReq.Merge.StrategiesAllowed) > 0 { // Note: Empty allowed strategies list means all are allowed
@ -178,7 +168,6 @@ func (v DefComments) Validate() error {
type DefStatusChecks struct {
RequireUIDs []string `json:"require_uids,omitempty"`
AllMustSucceed bool `json:"all_must_succeed,omitempty"`
}
func (v DefStatusChecks) Validate() error {