Skip to content

Commit 671c79d

Browse files
authored
Use GitHub's automerge instead of Mergify (#9126)
Co-authored-by: Andrew Murray <radarhere@users.noreply.github.com>
2 parents ca443f0 + cb1e81a commit 671c79d

5 files changed

Lines changed: 61 additions & 16 deletions

File tree

.github/mergify.yml

Lines changed: 0 additions & 13 deletions
This file was deleted.

.github/workflows/test-docker.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,12 @@ jobs:
108108
success:
109109
permissions:
110110
contents: none
111+
if: always()
111112
needs: build
112113
runs-on: ubuntu-latest
113114
name: Docker Test Successful
114115
steps:
115116
- name: Success
116-
run: echo Docker Test Successful
117+
env:
118+
BUILD_RESULT: ${{ needs.build.result }}
119+
run: test "$BUILD_RESULT" = "success"

.github/workflows/test-skipped.yml

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
# Report success for required status checks when the real test workflows are
2+
# skipped by their paths-ignore filters, so that, for example, PRs only touching
3+
# docs can still be merged under a ruleset requiring these checks. See
4+
# https://docs.github.com/en/pull-requests/how-tos/merge-and-close-pull-requests/troubleshooting-required-status-checks#handling-skipped-but-required-checks
5+
#
6+
# Keep the paths below in sync with the paths-ignore lists in test.yml,
7+
# test-docker.yml, test-mingw.yml and test-windows.yml.
8+
9+
name: Test skipped
10+
11+
on:
12+
pull_request:
13+
paths:
14+
- ".github/workflows/docs.yml"
15+
- ".github/workflows/wheels*"
16+
- ".gitmodules"
17+
- "docs/**"
18+
- "wheels/**"
19+
20+
permissions: {}
21+
22+
jobs:
23+
test:
24+
runs-on: ubuntu-slim
25+
name: Test Successful
26+
steps:
27+
- name: Success
28+
run: echo Tests not required for this change
29+
30+
test-docker:
31+
runs-on: ubuntu-slim
32+
name: Docker Test Successful
33+
steps:
34+
- name: Success
35+
run: echo Tests not required for this change
36+
37+
test-mingw:
38+
runs-on: ubuntu-slim
39+
name: MinGW
40+
steps:
41+
- name: Success
42+
run: echo Tests not required for this change
43+
44+
test-windows:
45+
runs-on: ubuntu-slim
46+
name: Windows Test Successful
47+
steps:
48+
- name: Success
49+
run: echo Tests not required for this change

.github/workflows/test-windows.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,9 +232,12 @@ jobs:
232232
success:
233233
permissions:
234234
contents: none
235+
if: always()
235236
needs: build
236237
runs-on: ubuntu-latest
237238
name: Windows Test Successful
238239
steps:
239240
- name: Success
240-
run: echo Windows Test Successful
241+
env:
242+
BUILD_RESULT: ${{ needs.build.result }}
243+
run: test "$BUILD_RESULT" = "success"

.github/workflows/test.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,9 +164,12 @@ jobs:
164164
success:
165165
permissions:
166166
contents: none
167+
if: always()
167168
needs: build
168169
runs-on: ubuntu-latest
169170
name: Test Successful
170171
steps:
171172
- name: Success
172-
run: echo Test Successful
173+
env:
174+
BUILD_RESULT: ${{ needs.build.result }}
175+
run: test "$BUILD_RESULT" = "success"

0 commit comments

Comments
 (0)