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

Prerelease ID #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Prerelease ID #102

wants to merge 1 commit into from

Conversation

scottux
Copy link

@scottux scottux commented Feb 2, 2015

Addressing #101

Allow for a preId to be set, semver.inc() uses this to prefix a prerelease bump so that 1.0.0 becomes 1.0.0-alpha.0 for instance.

Addressing geddski#101

Allow for a preId to be set, `semver.inc()` uses this to prefix a prerelease bump so that 1.0.0 becomes 1.0.0-alpha.0 for instance.
@drublic
Copy link
Collaborator

drublic commented Feb 3, 2015

Same here, great work. Thanks.
One thing bugs me a bit: if we need to set an option for a pre-release the
config might need updates across branches if you work with gitflow and do
pre releases and rcs on your development or release branch.
This would mean we need to maintain this in all branches.

Maybe we can investigate the change to a CLI flag: grunt release:prerelease
--alpha
Or something like that.

What do you think?

On Mon, Feb 2, 2015, 6:38 PM Scott [email protected] wrote:

Addressing #101 #101

Allow for a preId to be set, semver.inc() uses this to prefix a

prerelease bump so that 1.0.0 becomes 1.0.0-alpha.0 for instance.

You can view, comment on, or merge this pull request online at:

#102
Commit Summary

  • Prerelease ID

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#102.

@scottux
Copy link
Author

scottux commented Feb 3, 2015

I like that idea, I am always tagging off of master so this was better for me than having a separate branch for -rc.

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

Successfully merging this pull request may close these issues.

2 participants