Skip to content

Commit 0905668

Browse files
authored
ci: use check-success instead of status-success for GitHub Actions (#35188)
### Reason for this change After removing the CodeBuild requirement from Mergify in #35174, automatic PR merging stopped working. Mergify was looking for status-success=validate-pr and status-success=build, but GitHub Actions creates check runs, not statuses. CodeBuild used the GitHub Status API, while GitHub Actions uses the GitHub Checks API. ### Description of changes Changed all Mergify conditions from status-success=validate-pr and status-success=build to check-success=validate-pr and check-success=build to match what GitHub Actions actually creates. The previous configuration worked because Mergify was only waiting for the CodeBuild status (which was correctly configured), while the GitHub Actions conditions were silently failing but not blocking merges. ### Describe any new or updated permissions being added NA ### Description of how you validated changes Analyzed GitHub API responses for recent PRs to confirm: • CodeBuild creates statuses via GitHub Status API • GitHub Actions creates check runs via GitHub Checks API • Verified that approved PRs have successful validate-pr and build check runs but no corresponding statuses ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent c524471 commit 0905668

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

.mergify.yml

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ queue_rules:
1717
- -approved-reviews-by~=author
1818
# This is important! It makes the PR Linter work.
1919
- "#changes-requested-reviews-by=0"
20-
- status-success=validate-pr
21-
- status-success=build
20+
- check-success=validate-pr
21+
- check-success=build
2222
commit_message_template: |-
2323
{{ title }} (#{{ number }})
2424
{{ body }}
@@ -37,8 +37,8 @@ queue_rules:
3737
- -approved-reviews-by~=author
3838
# This is important! It makes the PR Linter work.
3939
- "#changes-requested-reviews-by=0"
40-
- status-success=validate-pr
41-
- status-success=build
40+
- check-success=validate-pr
41+
- check-success=build
4242
commit_message_template: |-
4343
{{ title }} (#{{ number }})
4444
{{ body }}
@@ -56,8 +56,8 @@ queue_rules:
5656
- -approved-reviews-by~=author
5757
# This is important! It makes the PR Linter work.
5858
- "#changes-requested-reviews-by=0"
59-
- status-success=validate-pr
60-
- status-success=build
59+
- check-success=validate-pr
60+
- check-success=build
6161
commit_message_template: |-
6262
{{ title }} (#{{ number }})
6363
{{ body }}
@@ -88,8 +88,8 @@ pull_request_rules:
8888
- -approved-reviews-by~=author
8989
# This is important! It makes the PR Linter work.
9090
- "#changes-requested-reviews-by=0"
91-
- status-success=validate-pr
92-
- status-success=build
91+
- check-success=validate-pr
92+
- check-success=build
9393
- name: automatic priority merge
9494
actions:
9595
comment:
@@ -109,8 +109,8 @@ pull_request_rules:
109109
- -approved-reviews-by~=author
110110
# This is important! It makes the PR Linter work.
111111
- "#changes-requested-reviews-by=0"
112-
- status-success=validate-pr
113-
- status-success=build
112+
- check-success=validate-pr
113+
- check-success=build
114114
- name: automatic merge (2+ approvers)
115115
actions:
116116
comment:
@@ -130,8 +130,8 @@ pull_request_rules:
130130
- -approved-reviews-by~=author
131131
# This is important! It makes the PR Linter work.
132132
- "#changes-requested-reviews-by=0"
133-
- status-success=validate-pr
134-
- status-success=build
133+
- check-success=validate-pr
134+
- check-success=build
135135
- name: automatic merge (no-squash)
136136
actions:
137137
comment:
@@ -151,8 +151,8 @@ pull_request_rules:
151151
- -approved-reviews-by~=author
152152
# This is important! It makes the PR Linter work.
153153
- "#changes-requested-reviews-by=0"
154-
- status-success=validate-pr
155-
- status-success=build
154+
- check-success=validate-pr
155+
- check-success=build
156156
- name: remove stale reviews
157157
actions:
158158
dismiss_reviews:
@@ -191,8 +191,8 @@ pull_request_rules:
191191
- "#approved-reviews-by>=1"
192192
# This is important! It makes the PR Linter work.
193193
- "#changes-requested-reviews-by=0"
194-
- status-success=validate-pr
195-
- status-success=build
194+
- check-success=validate-pr
195+
- check-success=build
196196
priority_rules:
197197
- name: priority for queue `default-merge`
198198
conditions:

0 commit comments

Comments
 (0)