RepoMirrors/go-ceph
1
0
Fork 0
mirror of https://github.com/ceph/go-ceph synced 2025-02-11 16:07:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

Include all status checks explicitly in the Mergify configuration

Browse Source 
Unfortunately it is not recommended to use a regular expression to check
for the CI status of multiple tests. In case some tests do not run and
fail to report the status, Mergify will not know something is wrong and
PRs might get merged.

The only way to make sure all status checks have passed successfully, is
to list them all separately.

See-also: https://docs.mergify.io/conditions.html#validating-all-status-check
Signed-off-by: Niels de Vos <ndevos@redhat.com>
This commit is contained in:
Niels de Vos 2020-11-03 15:09:03 +01:00 committed by John Mulligan
parent c7bab1c25c
commit 92b10d8699
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
.github/mergify.yml vendored
View File

@ -14,7 +14,11 @@ pull_request_rules:
- "#approved-reviews-by>=2"
- "#changes-requested-reviews-by=0"
- status-success=check
- status-success=test-suite
# each test should be listed separately, do not use regular expressions:
# https://docs.mergify.io/conditions.html#validating-all-status-check
- status-success=test-suite (mimic)
- status-success=test-suite (nautilus)
- status-success=test-suite (octopus)
actions:
merge:
method: rebase