Skip to content

Commit

Permalink
Propagate build failures (#363)
Browse files Browse the repository at this point in the history
Always run the `publish-build` step, regardless of whether the rosetta
pax/t5x build was attempted. This ensures that badges correctly reflect
build failures due to dependent builds failing.
  • Loading branch information
ashors1 authored Nov 8, 2023
1 parent a391095 commit f7a4f44
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/nightly-rosetta-pax-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ jobs:
publish-build:
needs: [metadata, build]
uses: ./.github/workflows/_publish_badge.yaml
if: ( success() || failure() ) && (github.event_name == 'workflow_run' && github.event.workflow_run.conclusion == 'success') || github.event_name == 'workflow_dispatch'
if: ( always() )
secrets: inherit
with:
ENDPOINT_FILENAME: 'rosetta-pax-build-status.json'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/nightly-rosetta-t5x-build-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ jobs:
publish-build:
needs: [metadata, build]
uses: ./.github/workflows/_publish_badge.yaml
if: ( success() || failure() ) && (github.event_name == 'workflow_run' && github.event.workflow_run.conclusion == 'success') || github.event_name == 'workflow_dispatch'
if: ( always() )
secrets: inherit
with:
ENDPOINT_FILENAME: 'rosetta-t5x-build-status.json'
Expand Down

0 comments on commit f7a4f44

Please sign in to comment.