-
-
Notifications
You must be signed in to change notification settings - Fork 26.9k
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
Bugs in extract-text-webpack-plugin #1668
Comments
Only related to WP2? Not for 0.9 branch? |
Only related to webpack-2 afaik, issues are tagged so and are related to wp2 features. |
This is worrying me. I need to start working on integrating the PWA stuff, and it seems like we'd have to backport it to 0.9.x since Webpack 2 is not ready 😞 |
I believe this never worked in WP 1 (with |
Full disclosure I haven't actually tested these issues to see if they're present, but since they're open, tagged, and +1'd I assume they're valid. |
OK I think we can live with this. |
At least webpack-contrib/extract-text-webpack-plugin#188 only happens in watch mode so it doesn’t affect us. |
@Timer / @gaearon - Anything like this where you have blocking / aggravating bugs with something living under |
Thanks for letting us know @d3viant0ne, we would love to see webpack-contrib/extract-text-webpack-plugin#406 fixed especially if we're going to turn on performance hints. |
@Timer - On the off chance you haven't seen it, @sokra is attacking the performance problem at it's source as opposed to trying to stick a bandaid on ExtractText. https://medium.com/webpack/the-new-css-workflow-step-1-79583bd107d7 More info as it becomes available :) |
Closing as stale. (Blockers have since been fixed.) |
blocking
These issues seem to break code that relies on
import()
andrequire
s css, which we now support. I have not tested this, but concerning nevertheless.would be nice
The text was updated successfully, but these errors were encountered: