Skip to content

update dependencies #677

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

Merged
merged 1 commit into from
Mar 3, 2017
Merged

update dependencies #677

merged 1 commit into from
Mar 3, 2017

Conversation

blade254353074
Copy link

When using yarn to install dependencies, yarn will lock the vue-hot-reload-api version to 2.0.1.
After update vue to 2.2.1...

image

So I updated the vue-loader dependencies ,run the test, and made this pr.

@xalexec
Copy link

xalexec commented Mar 1, 2017

+1

@kazupon kazupon requested a review from yyx990803 March 1, 2017 09:49
@kazupon
Copy link
Member

kazupon commented Mar 1, 2017

related #675

@LinusBorg
Copy link
Member

I don't think that all these dependency version bumps are necessary or related to this problem, and in case of inject-loader, we csn't use 3.0.0 because it doesn'T work correctly.

Can you limit the update to those packages that are problematic when updating with yarn?

@blade254353074
Copy link
Author

@LinusBorg My bad. I was too rash to update the dependencies...
Now I just update vue-hot-reload-api to 2.0.11.

@LinusBorg
Copy link
Member

/ping @yyx990803

A lot of people had problems getting this dependency to update correctly with yarn, even with deleting /node_modules, I think this should solve it.

@yyx990803 yyx990803 merged commit 6ec605d into vuejs:master Mar 3, 2017
kazupon pushed a commit to vuejs-templates/webpack that referenced this pull request Mar 3, 2017
xalexec pushed a commit to Transnal/vue-loader that referenced this pull request Mar 26, 2017
pksunkara pushed a commit to sunkarapk/spoiler that referenced this pull request Apr 7, 2017
wdev733 added a commit to wdev733/vue-dashboard that referenced this pull request Aug 6, 2019
fullstack412 added a commit to fullstack412/Vue-dashboard that referenced this pull request Aug 16, 2019
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.

5 participants