Skip to content

Commit

Permalink
ci: Remove summary steps
Browse files Browse the repository at this point in the history
We plan to adjust the way we mark workflows as required, and we will no
longer require the summary steps for that. Let's remove them to save
some CI cycles and shorten the list of status checks.

Signed-off-by: Quentin Monnet <[email protected]>
  • Loading branch information
qmonnet committed Nov 18, 2024
1 parent 02d7ce7 commit 764641d
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 32 deletions.
13 changes: 0 additions & 13 deletions .github/workflows/dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -153,16 +153,3 @@ jobs:
timeout-minutes: 60
with:
limit-access-to-actor: true

summary:
name: "summary"
runs-on: "ubuntu-latest"
needs:
- build
if: ${{ always() && needs.build.result != 'skipped' }}
steps:
- name: "Flag any build matrix failures"
if: ${{ needs.build.result != 'success' }}
run: |
>&2 echo "A critical step failed!"
exit 1
19 changes: 0 additions & 19 deletions .github/workflows/sterile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -228,22 +228,3 @@ jobs:
timeout-minutes: 60
with:
limit-access-to-actor: true

summary:
name: "summary"
runs-on: "ubuntu-latest"
needs:
- test
- push
if: ${{ always() && needs.test.result != 'skipped' && needs.push.result != 'skipped' }}
steps:
- name: "Flag any test failures"
if: ${{ needs.test.result != 'success' }}
run: |
>&2 echo "One or more required tests failed"
exit 1
- name: "Flag any push failures"
if: ${{ needs.push.result != 'success' }}
run: |
>&2 echo "One or more required pushes failed"
exit 1

0 comments on commit 764641d

Please sign in to comment.