-
Notifications
You must be signed in to change notification settings - Fork 71
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
Semantic release #44
Semantic release #44
Conversation
@@ -0,0 +1,4 @@ | |||
|
|||
cron_package: |
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.
great work @daks! but isn't it the logrotate formula? ;)
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.
This state is added for Suse platforms only at kitchen run, because Suse image does not seems to have cron installed. As logrotate directly depends on cron running, test fails :)
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.
@javierbertoli Would it be worth adding cron
to the Suse pre-salted images?
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.
Looking good, @daks. Just a few points:
- Need to add
bin/kitchen
. - Need to add
release-rules.js
. - What's the plan for
.kitchen.docker.yml
-- can this now be removed?
@myii new version with asked changes. |
Excellent, thanks @daks -- merged. Appreciate the reviews @aboe76 and @n-rodriguez. |
🎉 This PR is included in version 0.10.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
thanks everyone for the review and merge |
Implement semantic-release with a special test fix for Suse
os_family
.