Closed sokra closed 3 years ago
@sokra Thank you so much for whipping up an entire PR for Webpack 5. This is huge!
We are currently kinda heads down in the preparations for the Chrome Dev Summit 2020, so it will be a week or so before we get back to this. We were also previously discussing how we can keep both Webpack 4 and 5 available on tooling.report. But we’ll get back to that.
Again, thanks for putting in all this work!
We were also previously discussing how we can keep both Webpack 4 and 5 available on tooling.report. But we’ll get back to that.
I copied all existing webpack folders into a webpack-4
folder and the new webpack
folder is now webpack 5. Once you have the infrastructure you can use the webpack-4
folder in addition.
Apologies for the delay - I should be able to review this within the week!
Would be very useful to have this updated reference for Webpack 5 on https://bundlers.tooling.report/ - I fear if I hadn't checked the issue / pull request backlog for the project I may have made a different choice in a bundler. Looks like I am going to stick with webpack.
Anything I can help you with getting this reviewed?
@sokra can you enable "Allow edits and access to secrets by maintainers" for your PR?
@sokra can you enable "Allow edits and access to secrets by maintainers" for your PR?
It's already enabled.
I've skipped over all the webpack-4
tests since we agreed to simply remove them. There were no material changes when I diffed them against the webpack
directories currently on the dev
branch.
If it makes reviews easier we could delete those directories. For now I just ran this in DevTools to mark them as hidden (note: this will freeze the page for around 1 minute):
document.querySelectorAll('a[title*="/webpack-4/"]').forEach(n => n.parentNode.nextElementSibling.querySelector('input[type=checkbox]').click());
I've skipped over all the
webpack-4
tests since we agreed to simply remove them. There were no material changes when I diffed them against thewebpack
directories currently on thedev
branch.
ok. I removed the commit which copies them over.
Other than the few comments I made, this looks good to go. Thanks @sokra for putting this PR together, some great changes in webpack 5.
I'm pushing the merge button!
Publishing to live now. It'll be up in a couple of mins.
import "./image.png"
tonew URL("image.png", import.meta.url)
which is the new recommended style for assetsimport
this is covered by web specs and fully working without bundler (more future-proof)import()
. Partial because it's manual.