-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Update references to webpack 4 #2389
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor change and and its good
src/content/guides/caching.md
Outdated
Hash: 17c37ce65c84b8ed5eb8 | ||
Version: webpack 4.16.2 | ||
Time: 637ms | ||
Built at: 2018-07-24 18:32:46 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we dont include date into logs
src/content/guides/caching.md
Outdated
Hash: 70fb9e00dee0bada797d | ||
Version: webpack 4.16.2 | ||
Time: 875ms | ||
Built at: 2018-07-24 18:33:50 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
Thanks! Dates removed from logs 🙌 |
Thanks! |
Hi! 👋 This PR fixes #2296
Following is a list of removed resources and a brief explanation of the reasons behind:
Predictable Long Term Caching
Based on webpack 3 (i.e. code snippets use CommonsChunkPlugin)
Webpack & Caching
While some of the recommendations might still be useful, some other make use CommonsChunkPlugin. Maybe the reference could be updated and then kept in the docs?
Advanced Webpack Presentation
Based on webpack 3 (i.e. code snippets use CommonsChunkPlugin)
Issue 1315
Issue created on 2015 and closed this May. Might be relevant to webpack 4 to a certain extent, but does not provide valid answers.
This is my first PR, so all feedback is really appreciated. Thanks for your patience!