Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Do not download a pinned version of cli if it is already installed. #1630

Merged
merged 1 commit into from
Mar 1, 2016

Conversation

moozzyk
Copy link

@moozzyk moozzyk commented Mar 1, 2016

Fixes #1628

@TheRealPiotrP
Copy link

@krwq I'm ok with merging this after CR, but please be aware that the scripts are currently being entirely rewritten per #1482

@krwq
Copy link
Member

krwq commented Mar 1, 2016

Looks good to me! As Peter mentioned this will change soon

moozzyk added a commit that referenced this pull request Mar 1, 2016
Do not download a pinned version of cli if it is already installed.
@moozzyk moozzyk merged commit ab5df03 into rel/1.0.0 Mar 1, 2016
@leecow leecow removed the in progress label Mar 1, 2016
@moozzyk
Copy link
Author

moozzyk commented Mar 1, 2016

Thanks!

@eerhardt eerhardt deleted the pawelka/pinned branch May 4, 2016 22:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants