Skip to content

Commit c45b0bd

Browse files
authored
ci: aggregate success results into a single job (#483)
This greatly simplifies the terraform configuration, because we only have to require the "Success" status check. It’s simpler than listing all the checks in terraform which can be sometimes problematic: - We need to list all the job names generated using `matrix` - We need to open a PR in infrastructure-as-code AND to deploy the change when we change / add / remove a job Here, we just list the required jobs directly as dependencies of the `success` job, without having to modify multiple repositories. Job dependencies also does not use the job names, but the job "key", so there is much less things to enumerate. Less error prone overall.
1 parent dfbe947 commit c45b0bd

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

.github/workflows/ci.yml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,3 +154,18 @@ jobs:
154154

155155
- name: Lock files
156156
run: cargo xtask check locks -v
157+
158+
success:
159+
name: Success
160+
runs-on: ubuntu-latest
161+
if: ${{ success() }}
162+
needs:
163+
- formatting
164+
- typos
165+
- checks
166+
- fuzz
167+
- web
168+
169+
steps:
170+
- name: CI succeeded
171+
run: exit 0

0 commit comments

Comments
 (0)