-
Notifications
You must be signed in to change notification settings - Fork 22
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
ci(github): simplify release.yaml #3383
base: master
Are you sure you want to change the base?
Conversation
Remove some indirections using branches for checkout when we have a SHA and extract params as env vars. This also makes the workflows more secure Signed-off-by: Charly Molter <[email protected]>
✅ Deploy Preview for kuma-gui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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, thanks!
Signed-off-by: Charly Molter <[email protected]>
Signed-off-by: Charly Molter <[email protected]>
(FYI you hit a flake here we picked up last week #3376 was hoping to get to it today, I just re-ran it) |
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 👍
Remove some indirections using branches for checkout when we have a SHA and extract params as env vars.
This also makes the workflows more secure