-
Notifications
You must be signed in to change notification settings - Fork 38
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
css-loader 4 compatibility requires documentation #46
Comments
OK I think I found it: https://github.com/webpack-contrib/css-loader/blob/master/CHANGELOG.md#-breaking-changes
I changed the local package in my const url = '!!' + iconFontLoaderPath + '?' + JSON.stringify(options) + '!~' + iconFontPlaceholderPath; And it worked successfully. |
Hey @davidhouweling you are right iconfont-webpack-plugin/lib/postcss-plugin.js Line 161 in 337f38c
is causing the issues.. I am happy that the css-loader is finally removing the |
css-loader@4 introduced a breaking change which meant the use of `~` needs to be relocated from before the `!!` to after the loader use + `!` BREAKING CHANGE: this change requires css-loader@4 to work, earlier versions of css-loader should use previous version re #46
css-loader@4 introduced a breaking change which meant the use of `~` needs to be relocated from before the `!!` to after the loader use + `!` BREAKING CHANGE: this change requires css-loader@4 to work, earlier versions of css-loader should use previous version re #46
I've attempted to upgrade to to css-loader 4 in our local project but ran into the following error
I tracked it down to
iconfont-webpack-plugin/lib/postcss-plugin.js
Line 161 in 337f38c
I thought the way to fix it is to tell
css-loader@4
not to interpret~!!
:However this makes the url not parse at all. Any idea how to solve this?
The text was updated successfully, but these errors were encountered: