Skip to content
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 an automated changelog #91

Merged
merged 1 commit into from
Jun 3, 2019

Conversation

daks
Copy link
Member

@daks daks commented May 29, 2019

WIP based on previous work. May need updates.

Very basic tests (same as before) but they pass on all platforms! :) https://travis-ci.com/daks/postfix-formula/builds/113663324

@daks daks requested review from javierbertoli and myii May 29, 2019 21:31
Copy link
Member

@myii myii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another milestone PR, great work! A few changes inline and a few more suggestion here:

  1. Still needs:
    1. bin/kitchen
    2. test/integration/default/inspec.yml
  2. Directory structure (standard locations):
    1. test/integration/default/pkgs_spec.rb => test/integration/default/controls/pkgs_spec.rb
    2. test/integration/default/service_spec.rb => test/integration/default/controls/service_spec.rb

docs/README.rst Outdated Show resolved Hide resolved
docs/README.rst Outdated
``bin/kitchen converge``
^^^^^^^^^^^^^^^^^^^^^^^^

Creates the docker instance and runs the main(s) state(s), ready for testing.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I usually check kitchen.yml to see which states are being run. But something like this could be a better solution in the long-term, by fixing it in the template-formula. Not really main(s) state(s), though. Perhaps main state(s) is good enough.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, main state(s) is more correct

@daks
Copy link
Member Author

daks commented Jun 3, 2019

changes asked pushed.

@myii myii merged commit d89b07c into saltstack-formulas:master Jun 3, 2019
@myii
Copy link
Member

myii commented Jun 3, 2019

Excellent, thanks @daks -- merged.

@saltstack-formulas-travis

🎉 This PR is included in version 0.17.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants