-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fail run upon error #28
Fail run upon error #28
Conversation
39bf1f9
to
7be3c5b
Compare
I have successfully added a CI job that is meant to fail, and only succeeds if it fails. |
This PR is currently blocking for other PRs, since the added CI job has been made a requirement for merging into |
This is no longer true, however, upon merging this PR, the branch protection should be updated to include the new CI job. |
@ml-evs this is ready for review whenever :) |
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.
LGTM @CasperWA, thanks.
- name: This runs ONLY if 'Run action' doesn't fail as expected | ||
if: steps.action.outcome != 'failure' || steps.action.conclusion != 'success' | ||
run: | | ||
echo "Outcome: ${{ steps.action.outcome }} (not 'failure'), Conclusion: ${{ steps.action.conclusion }} (not '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.
Nice!
Fixes #25
set -e
has been added toentrypoint.sh
, so that any non-zero return value will stop the script.Furthermore, an extra test for an invalid version value has been added.