-
Notifications
You must be signed in to change notification settings - Fork 916
vue-loader breaks since Webpack 2.1.0-beta.26 #464
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
Comments
In the loader.js the line 80 the following horrible hack solves the problem: Naturally this is not a proposed fix, merely a step to pin down the issue. |
An actual proposal here for a fix: Edit: the same idea seems to be necessary for 'script' and 'template cases' as well in getLoaderString |
…heir full name (vuejs#464) * In Webpack 2 loaders have to be explicitly declared, the "-loader" part of their name is no longer implied.
I had this problem using A fix should be tested for pug/jade support in templates, as it seems these needs the normal modules (i.e. 'pug') and not the loader module ('pug-loader') |
I' m using Webpack 1.13.3 (npm version) |
@thorning, I tested the "fix" with both Jade and Pug. For templates "raw" changes to "raw-loader" the engine query param stays the same. See: @ylhuang0423, Breaking |
@riovir, Maybe try...catch to require loaders (like what Yet it's not a perfect solution for some special cases. |
Can you try 9.9.3? |
9.9.3 works well with Webpack 2 beta 26 as far as I managed to verify, thank you. |
https://github.com/webpack/webpack/releases/tag/v2.1.0-beta.26
The breaking change now forces -loader to be used. Since vue-loader internally also uses loaders the problem might be caused by this change. See:
Error: Module '....../node_modules/less/index.js' is not a loader
The text was updated successfully, but these errors were encountered: