-
Notifications
You must be signed in to change notification settings - Fork 8
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
versionist support: add config and changelog file #9
Conversation
9317884
to
15bce0f
Compare
@hedss when this PR will look good, can you enable VersionBot on this repo? (and then this could be the first PR merged by VersionBot itself, do I get that right?) |
Change-type: patch
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!
' - {{capitalize this.subject}} [{{this.author}}]', | ||
'{{else}}', | ||
'* {{capitalize this.subject}}', | ||
'{{/if}}', |
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.
I've just noticed that there's an inconsistency here - the 'if' part does - {{capitalize this.subject}}
but the 'else' part does * {{capitalize this.subject}}
.
And compare it to https://github.com/resin-io/versionist/blob/master/versionist.conf.js which uses just *
and not -
.
module.exports = { | ||
// This setup allows the editing and parsing of footer tags to get version and type information, | ||
// as well as ensuring tags of the type 'v<major>.<minor>.<patch>' are used. | ||
// It increments in a semver compatible fashion and does not do ano NPM package info update (N/A). |
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.
ano -> any ?
Second part of #7 adding versionist support so VersionBot will know what to do, when enabled in this repo