-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[tests] Fix summary workflow to always run #10048
Conversation
|
.github/workflows/test.yml
Outdated
steps: | ||
- name: Done | ||
run: echo "Done." | ||
- run: |- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add a name? Currently it shows as "Run for status in failure" or "Run for status in success"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 2ff02a8
Description
This ensures the Summary job always runs to check if any test failed and cause itself to fail as well.
This allows us to dynamically add as many concurrent jobs as we want but only have one (Summary) marked as required.
Testing
I verified this worked by making a change to
@vercel/static-config
which is not required and the Summary check failed:https://github.com/vercel/vercel/actions/runs/5136560652/jobs/9243924596
Then I reverted it so that the test was passing and the Summary check passed:
https://github.com/vercel/vercel/actions/runs/5137401343/jobs/9245554297