-
Notifications
You must be signed in to change notification settings - Fork 363
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
feat(semantic-release): implement for this formula #220
Conversation
a2dbafa
to
c97b956
Compare
Included latest changes from saltstack-formulas/template-formula#148. |
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.
Hi @myii nice job! LGTM.
c97b956
to
e670547
Compare
* Include latest pre-salted images replacing EOL platforms - Ref: saltstack-formulas/template-formula#148
e670547
to
1d2e2f5
Compare
Now using the new |
@myii are you ok with merging? |
@aboe76 It's done from my end but @daks raised some concerns in the |
@myii I think you are right, merging it. |
🎉 This PR is included in version 0.50.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Thanks @aboe76. |
This PR introduces
semantic-release
, as we've already done with many other formulas. The functionality has already been tested in my fork. There are a number of issues I've encountered when preparing this PR but I will have to cover them in subsequent comments when I get the opportunity for that. In the meantime, I would like to confirm that we wantsemantic-release
implemented for this formula.CC: @javierbertoli @aboe76 @noelmcloughlin @alxwr.