ci: do_not_merge: pass instead of skip on mergeable PRs
Change the workflow to pass instead of skip if the PR is mergeable, just trying to see if it helps with PRs not appearing as mergeable because they do not match status:success when they should. Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
This commit is contained in:
parent
0617bd306a
commit
fa8ebd3e45
1 changed files with 4 additions and 4 deletions
8
.github/workflows/do_not_merge.yml
vendored
8
.github/workflows/do_not_merge.yml
vendored
|
|
@ -6,14 +6,14 @@ on:
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
do-not-merge:
|
do-not-merge:
|
||||||
if: ${{ contains(github.event.*.labels.*.name, 'DNM') ||
|
|
||||||
contains(github.event.*.labels.*.name, 'TSC') ||
|
|
||||||
contains(github.event.*.labels.*.name, 'Architecture Review') ||
|
|
||||||
contains(github.event.*.labels.*.name, 'dev-review') }}
|
|
||||||
name: Prevent Merging
|
name: Prevent Merging
|
||||||
runs-on: ubuntu-22.04
|
runs-on: ubuntu-22.04
|
||||||
steps:
|
steps:
|
||||||
- name: Check for label
|
- name: Check for label
|
||||||
|
if: ${{ contains(github.event.*.labels.*.name, 'DNM') ||
|
||||||
|
contains(github.event.*.labels.*.name, 'TSC') ||
|
||||||
|
contains(github.event.*.labels.*.name, 'Architecture Review') ||
|
||||||
|
contains(github.event.*.labels.*.name, 'dev-review') }}
|
||||||
run: |
|
run: |
|
||||||
echo "Pull request is labeled as 'DNM', 'TSC', 'Architecture Review' or 'dev-review'."
|
echo "Pull request is labeled as 'DNM', 'TSC', 'Architecture Review' or 'dev-review'."
|
||||||
echo "This workflow fails so that the pull request cannot be merged."
|
echo "This workflow fails so that the pull request cannot be merged."
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue