Skip to content

Commit

Permalink
ci: do not let Mergify remove the ok-to-test label
Browse files Browse the repository at this point in the history
Sometimes Mergify removed the `ok-to-test` label before the Pull Request
Commentor action have been run. With the updated commentor action, the
`ok-to-test` label is removed after leaving comments. There is no need
for Mergify to remove the label anymore.

Signed-off-by: Niels de Vos <ndevos@redhat.com>
  • Loading branch information
nixpanic committed Feb 20, 2023
1 parent 3252c1e commit 56c8f1e
Showing 1 changed file with 0 additions and 18 deletions.
18 changes: 0 additions & 18 deletions .mergify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,6 @@ pull_request_rules:
message: "This pull request now has conflicts with the target branch.
Could you please resolve conflicts and force push the corrected
changes? 🙏"
label:
remove:
- ok-to-test

- name: update dependencies by dependabot (skip commitlint)
conditions:
Expand Down Expand Up @@ -108,19 +105,13 @@ pull_request_rules:
queue:
name: default
delete_head_branch: {}
label:
remove:
- ok-to-test

- name: dismiss review of merged pull request
conditions:
- base~=^(devel)|(release-.+)$
- merged
actions:
dismiss_reviews: {}
label:
remove:
- ok-to-test

- name: automatic merge
conditions:
Expand Down Expand Up @@ -153,9 +144,6 @@ pull_request_rules:
queue:
name: default
delete_head_branch: {}
label:
remove:
- ok-to-test

- name: automatic merge PR having ready-to-merge label
conditions:
Expand Down Expand Up @@ -187,9 +175,6 @@ pull_request_rules:
queue:
name: default
delete_head_branch: {}
label:
remove:
- ok-to-test

- name: backport patches to release-v3.6 branch
conditions:
Expand Down Expand Up @@ -247,9 +232,6 @@ pull_request_rules:
queue:
name: default
delete_head_branch: {}
label:
remove:
- ok-to-test

##
## Automatically set/remove labels
Expand Down

0 comments on commit 56c8f1e

Please sign in to comment.