Skip to content

Commit 62378a9

Browse files
ci: Build required check includes merge gate (#6728)
The ready-to-merge-gate wasn't included in the build-required-check which allowed merging PRs without all the required checks. This is fixed now.
1 parent 5ef2eed commit 62378a9

File tree

4 files changed

+5
-0
lines changed

4 files changed

+5
-0
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -401,6 +401,7 @@ jobs:
401401
check-debug-with-UIKit,
402402
check-release-with-UIKit,
403403
check-compiling-async-safe-logs,
404+
ready-to-merge-gate,
404405
]
405406
name: Build
406407
# This is necessary since a failed/skipped dependent job would cause this job to be skipped

.github/workflows/release.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -384,6 +384,7 @@ jobs:
384384
swift-build,
385385
duplication-tests,
386386
app-metrics,
387+
ready-to-merge-gate,
387388
]
388389
name: Release
389390
# This is necessary since a failed/skipped dependent job would cause this job to be skipped

.github/workflows/test.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ jobs:
4848
with:
4949
token: ${{ github.token }}
5050
filters: .github/file-filters.yml
51+
5152
build-test-server:
5253
name: Build test server
5354
if: needs.files-changed.outputs.run_unit_tests_for_prs == 'true'
@@ -394,6 +395,7 @@ jobs:
394395
build-test-server,
395396
distribution-tests,
396397
unit-tests,
398+
ready-to-merge-gate,
397399
unit-tests-with-test-server,
398400
]
399401
name: Unit Tests

.github/workflows/ui-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,7 @@ jobs:
218218
ui-tests-swift-ui,
219219
ui-tests-swift,
220220
ui-tests-swift6,
221+
ready-to-merge-gate,
221222
]
222223
name: UI Tests
223224
# This is necessary since a failed/skipped dependent job would cause this job to be skipped

0 commit comments

Comments
 (0)