-
Notifications
You must be signed in to change notification settings - Fork 995
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
Continue on error for the composite actions #1763
Continue on error for the composite actions #1763
Conversation
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.
We need to make sure we remove secrets from the allowed contexts before merging this
Co-authored-by: Ferenc Hammerl <[email protected]>
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, Could you please link to an example test workflow displaying a successful run?
Example workflow file:
Inside the composite action:
Failing composite action:
|
@thboop Are the changes you requested done? I would really like to use this feature 😄 |
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
@thboop is this feature available with your merge from above or does it require an additional deployment? |
Any idea when this will be released? the latest release was two days before this was merged |
Hi all, which version of GH enterprise has this feature? |
Fixes #1457