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

git-flow versiontag #5

Closed
matt328 opened this issue Jul 5, 2016 · 5 comments
Closed

git-flow versiontag #5

matt328 opened this issue Jul 5, 2016 · 5 comments
Assignees

Comments

@matt328
Copy link
Contributor

matt328 commented Jul 5, 2016

If I specify a versiontag in my git-flow options, I have noticed two issues:

  1. The versiontag is prepended to the version written into my package.json. I don't think that should be.
  2. The tag created in git has the versiontag prepended twice. ex: if my versiontag is 'REL-', my git tags look like 'REL-REL-x.x.x'. I don't think this should be either, it is probably a side effect of prepending the versiontag and putting that value in options.next_version.
@mrkmg mrkmg self-assigned this Jul 5, 2016
@matt328
Copy link
Contributor Author

matt328 commented Jul 5, 2016

After looking at this some more, I'm not sure if node-generate-release should even bother reading the versiontag property, provided of course that the current behavior of prepending it in package.json is not intended. git-flow will prepend it to the tag name itself, all you have to supply to git-flow is the 'release name'.

I have this working in my fork, and thanks to 'npm link' I am able to release my project. Let me know if you would be interested in a PR.

@mrkmg
Copy link
Owner

mrkmg commented Jul 5, 2016

Would love a PR!

@matt328
Copy link
Contributor Author

matt328 commented Jul 5, 2016

PR submitted which just removes versiontag altogether, although after considering usage outside of NodeJS projects, I might need to think about that some more. I don't think it is valid to have prefixes on your version in package.json, but that isn't to say other types of projects won't want access to the prefixed version string for writing into tokenized files. Your thoughts?

@mrkmg
Copy link
Owner

mrkmg commented Jul 5, 2016

I will do some research in other package systems. Obviously it needs to change for NPM at least, so I will mostly likely accept the PR and then if needed add in special circumstances for other types of packages.

@mrkmg
Copy link
Owner

mrkmg commented Jul 5, 2016

It appears that we really do not need the version tag for the couple things I looked into. Merged the PR.

@mrkmg mrkmg closed this as completed Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants