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
Ignore bower.json "version" field in bower version
#2142
Conversation
74eeaee
to
77a84e8
Compare
Also, if this change is too drastic, an alternative approach could be to update the spec and un-deprecate the "version" field. I think we should definitely do one of these two things - currently, the spec is not consistent with Bower's behaviour. |
Pinging @bower/contributors |
I think it's good to merge, but we should use |
`bower version` currently takes the version from the "version" field of bower.json, but the spec says that bower ignores it. This commit updates the `bower version` tests so that they are consistent with the spec. Now, the tests are based on the assumption that `bower version` should only use git tags.
Fixes bower#2139. Make `bower version` operate solely on git tags, ignoring the version field in bower.json (if it exists).
77a84e8
to
49c7fef
Compare
Done; how's this? |
Close, but it doesn't preserve |
Also --help documentation is not updated |
Possible PR successor at #2232 |
Fixes #2139. See the commits for a description of the changes. I just went ahead and did this, because it was easy.
I was thinking that it might be a good idea to warn if the "version" field exists, too. If you agree, where should that go?
Also, if bower now uses tags for versions, I suppose we should support svn too?