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

Add tests for the update command #283

Closed
wibblymat opened this issue Feb 27, 2013 · 4 comments
Closed

Add tests for the update command #283

wibblymat opened this issue Feb 27, 2013 · 4 comments
Labels

Comments

@wibblymat
Copy link
Member

To catch issues like #282

@necolas
Copy link
Contributor

necolas commented Mar 9, 2013

@satazor Is update only meant to update to the latest patch release when you've specified a semver with explicit major and minor? It's not a very clear what this command is for.

@satazor
Copy link
Member

satazor commented Mar 11, 2013

@necolas both install and update commands are ambiguous as they stand now.

Will quote something that I emailed in the last bower meeting:

The update command updates to the latest patch release, which is kind of incorrect if the user specified a major or minor range.

@necolas
Copy link
Contributor

necolas commented Mar 11, 2013

Thanks @satazor. Looking at the functionality of update in other package managers, Bower's felt confusing. Good to know that it actually is confusing!

@satazor
Copy link
Member

satazor commented Jul 10, 2013

The update on the rewrite works great now! Also tests are in the todo list!

@satazor satazor closed this as completed Jul 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

3 participants