diff --git a/.noir-sync-commit b/.noir-sync-commit index 3cf17e0a3a3..b64f398c068 100644 --- a/.noir-sync-commit +++ b/.noir-sync-commit @@ -1 +1 @@ -3b8d1da2ac9f751b8fff3eaad0c64cbba8d01575 +9471e28ad6f02bf2fae3782c3db68106b615595f diff --git a/noir/noir-repo/.github/workflows/formatting.yml b/noir/noir-repo/.github/workflows/formatting.yml index b132ba6f938..4e836ef2493 100644 --- a/noir/noir-repo/.github/workflows/formatting.yml +++ b/noir/noir-repo/.github/workflows/formatting.yml @@ -143,3 +143,29 @@ jobs: - name: Format test suite working-directory: ./test_programs run: ./format.sh check + + # This is a job which depends on all test jobs and reports the overall status. + # This allows us to add/remove test jobs without having to update the required workflows. + formatting-end: + name: Formatting End + runs-on: ubuntu-22.04 + # We want this job to always run (even if the dependant jobs fail) as we want this job to fail rather than skipping. + if: ${{ always() }} + needs: + - clippy + - rustfmt + - eslint + - nargo_fmt + + steps: + - name: Report overall success + run: | + if [[ $FAIL == true ]]; then + exit 1 + else + exit 0 + fi + env: + # We treat any skipped or failing jobs as a failure for the workflow as a whole. + FAIL: ${{ contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') || contains(needs.*.result, 'skipped') }} +