-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update to webpack 3 #707
Update to webpack 3 #707
Conversation
Thank you for that PR :) Good call on Now we just have to wait for: webpack-contrib/extract-text-webpack-plugin#540 and we will be good to go! |
@mastilver - |
@d3viant0ne I can't seem to find |
@filipesilva it's |
Ah, apologies, I misunderstoord 👍 |
@yuffiy Do you mind updating your PR to get the latest I'm unable to do it atm |
@yuffiy @d3viant0ne Thank you for your work! 👍 @jantimon I let you squash, merge and release that :) |
It's full backwards compatible so we can just release it as 2.29 right? |
@jantimon Yep, no tests for webpack@1/2 have been changed, so I would think there is not breaking changes |
Why did you remove the following files? |
Thank you so much @d3viant0ne @yuffiy @mastilver for your work :) released as 2.29 |
Should this fix the |
Working combo for me:
|
it's very helpful to me! |
Thanks all |
Just a heads up it'd be good to add a git release tag so it shows up in the github releases page. Took me a while to realise 2.28.0 isn't the latest and was confused for a while! :) Thanks for the work on this great plugin! |
@chrisui done |
@mastilver ⭐️ |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
closes #699
closes #705