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: make tagPrefix configurable #143

Merged
merged 4 commits into from
Nov 27, 2016

Conversation

niieani
Copy link
Contributor

@niieani niieani commented Nov 20, 2016

At Aurelia we don't use any tag prefixes, our tags are just 1.0, thus we couldn't use standard-version as is. Right now standard-version has a hardcoded v prefix for all the git tags for releases, i.e. v1.0, v1.1. This change makes it configurable so you can have no prefix or a custom prefix if you wish. I left the default v prefix, so this is a non-breaking change.

Copy link
Member

@bcoe bcoe left a comment

Choose a reason for hiding this comment

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

It goes slightly against the stated goal of making standard-version relatively configuration free, but I appreciate that this will allow us to support a wider variety of projects.

Get the tests passing and we'll land this 👍

.option('tag-prefix', {
alias: 't',
describe: 'Set a custom prefix for the git tag to be created',
type: 'boolean',
Copy link
Member

Choose a reason for hiding this comment

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

If you switch the type to string rather than boolean I believe the tests should start passing.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for feedback! I didn't notice the test failed, sorry about that. Corrected now, but I see merge conflict. I'll see if I can rebase.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2925833 on niieani:patch-1 into 5c543a1 on conventional-changelog:master.

1 similar comment
@coveralls
Copy link

coveralls commented Nov 27, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 2925833 on niieani:patch-1 into 5c543a1 on conventional-changelog:master.

@bcoe bcoe merged commit 70b20c8 into conventional-changelog:master Nov 27, 2016
@niieani niieani deleted the patch-1 branch November 27, 2016 00:23
@bcoe
Copy link
Member

bcoe commented Nov 27, 2016

@niieani I've landed your changes in an upcoming release, would love an extra set of eyes on it. To test it out:

npm cache clear; npm i standard-version@next

👍

@niieani
Copy link
Contributor Author

niieani commented Nov 27, 2016

Many thanks! I'll have a go at it.

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

Successfully merging this pull request may close these issues.

3 participants