-
Notifications
You must be signed in to change notification settings - Fork 140
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
Version the spec itself #48
Comments
I think instead of that we could introduce
|
To clarify, are you suggesting using the Bower CLI tool's version number in The problem with the former is there's (vexingly) no good correspondence between the Bower CLI tool and the spec. Also, e.g. if the Bower CLI tool added a bunch of wizards for doing things, that would necessitate a minor version bump, but bumping the |
I mean version of bower used (not the spec). As you said, there's not enough correspondence between bower and spec, so it's safer to require given bower version than the version of spec. Spec could define what bower version implements what (if any version implements it at all). I don't think we need wizards for that. |
|
I'm working on custom resolvers feature, so I guess we could include them as well.. |
The spec itself ought to be versioned, and a field should be added to
bower.json
indicating at least the major (and perhaps also the minor) version of thebower.json
spec that it claims to conform to.This would allow for an easier migration/upgrade path when backward-incompatible changes need to be made to the spec. If a
bower.json
lacks abower.json
-spec-version-number, it should default to version "0" of the spec, for backward compatibility with old packages. Version 0 should be tagged retroactively as some commit circa the current version of the spec as of when this issue was filed. Abower.json
using an outdated major version of the spec should at least emit a warning, and at most fail with an error if it's so old that supporting it is too burdensome for Bower or is bad public policy for Bower.The text was updated successfully, but these errors were encountered: