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

Fix update command not installing missing packages #1519

Merged
merged 2 commits into from
Sep 16, 2014
Merged

Fix update command not installing missing packages #1519

merged 2 commits into from
Sep 16, 2014

Conversation

LaurentGoderre
Copy link
Contributor

No description provided.

@LaurentGoderre LaurentGoderre changed the title DO NOT MERGE: Fix update copmmand not installing missing packages DO NOT MERGE: Fix update command not installing missing packages Sep 15, 2014
@LaurentGoderre
Copy link
Contributor Author

@sheerun this addresses the immediate problem. If you agree with the change, can we merge and bump? I will add the rest of the test after the bump.

@LaurentGoderre LaurentGoderre changed the title DO NOT MERGE: Fix update command not installing missing packages Fix update command not installing missing packages Sep 16, 2014
@LaurentGoderre
Copy link
Contributor Author

@sheerun this is ready to go if you are ok with it. It has all the base test too

@sheerun
Copy link
Contributor

sheerun commented Sep 16, 2014

Thank you, embarrassing mistake :)

It'll try to release this fix as soon as possible.

sheerun added a commit that referenced this pull request Sep 16, 2014
Fix update command not installing missing packages
@sheerun sheerun merged commit ee6c483 into bower:master Sep 16, 2014
@LaurentGoderre LaurentGoderre deleted the fix-1518 branch September 16, 2014 20:48
@LaurentGoderre
Copy link
Contributor Author

@sheerun this kind of stuff is quite frequent I find. That's why we write tests, to protect us from ourselves ;)

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

Successfully merging this pull request may close these issues.

2 participants