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 bin not exist when installed via yarn #206

Merged
merged 2 commits into from
Oct 28, 2016
Merged

Fix bin not exist when installed via yarn #206

merged 2 commits into from
Oct 28, 2016

Conversation

CYBAI
Copy link

@CYBAI CYBAI commented Oct 22, 2016

yarn will link bin files in package.json.

This should fix #205 and #193.

Besides, due to yarn cached installed packages, if you have installed vue-cli before, please remove them from .yarn-cache and re-install it again.

@posva
Copy link
Member

posva commented Oct 24, 2016

@CYBAI Thanks!
About the last comment, wouldn't release a new version be enough?

@CYBAI
Copy link
Author

CYBAI commented Oct 24, 2016

@posva
oh! If releasing a new version, I think users will no need to do what I mentioned in last comment. 😄

@CYBAI
Copy link
Author

CYBAI commented Oct 25, 2016

Here is how I test locally

yarn install with original version (I've already installed vue-cli with yarn before, so it just installed only 1.54s)

→ yarn global add file:/path/to/vue-cli
yarn global v0.16.1
warning No license field
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 📃  Building fresh packages...
success Installed vue-cli@2.4.0 with binaries:
      - vue
✨  Done in 1.54s.

then update package.json for version and bin, it will install vue-init and vue-list successfully.

→ yarn global add file:/path/to/vue-cli
yarn global v0.16.1
warning No license field
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 📃  Building fresh packages...
success Installed vue-cli@2.4.999 with binaries:
      - vue
      - vue-init
      - vue-list
✨  Done in 24.01s.

@posva
Copy link
Member

posva commented Oct 27, 2016

@CYBAI Works well! Thanks for the tips btw!

@zigomir
Copy link
Contributor

zigomir commented Oct 28, 2016

@CYBAI thanks for this! I tested it and it works. Please bump version to 2.4.1 because without this change (as you mentioned) fix won't work (at least it hasn't for me). I know we'd bump version anyway, but since it's a requirement for this particular fix, I'd include it here.

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

Successfully merging this pull request may close these issues.

3 participants