Skip to content

Commit

Permalink
fix(ci): Always run merge-check (#6065)
Browse files Browse the repository at this point in the history
Reintroduces the fix from
#5873 which was
accidentally removed in
#5889
  • Loading branch information
spalladino authored Apr 26, 2024
1 parent ea030e5 commit b43b84f
Showing 1 changed file with 17 additions and 8 deletions.
25 changes: 17 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -210,15 +210,24 @@ jobs:
merge-check:
runs-on: ubuntu-latest
needs:
[
e2e,
bb-native-tests,
bb-bench,
yarn-project-formatting,
yarn-project-test,
]
- e2e
- bb-native-tests
- bb-bench
- yarn-project-formatting
- yarn-project-test
if: always()
steps:
- run: echo Pull request merging now allowed.
- run: |
echo "e2e status: ${{ needs.e2e.result }}"
echo "bb-native-tests status: ${{ needs.bb-native-tests.result }}"
echo "bb-bench status: ${{ needs.bb-bench.result }}"
echo "yarn-project-formatting status: ${{ needs.yarn-project-formatting.result }}"
echo "yarn-project-test status: ${{ needs.yarn-project-test.result }}"
if [[ "${{ needs.e2e.result }}" != 'success' || "${{ needs.bb-native-tests.result }}" != 'success' || "${{ needs.bb-bench.result }}" != 'success' || "${{ needs.yarn-project-formatting.result }}" != 'success' || "${{ needs.yarn-project-test.result }}" != 'success' ]]; then
echo "Pull request merging not allowed due to failures."
exit 1
fi
echo "Pull request merging now allowed."
notify:
needs:
Expand Down

0 comments on commit b43b84f

Please sign in to comment.