-
Notifications
You must be signed in to change notification settings - Fork 213
chore: Update dependency webpack to v5 #1674
Conversation
Hi! Thank you for the PR. In order to move this forwards:
|
Ported the CI changes from #1663. |
Thank you for looking at the CI changes. Could you move them to their own PR? I'm a fan of keeping PRs to be about one change at a time, and this PR is already pretty large (+1,104 −1,304) :-) |
Created a new PR with just the CI changes. #1675 |
It's possible some of the compilation problem I'm having will be solved when webpack-chain gets the webpack 5 support. |
The VueLoaderPlugin problem is eluding me. I'll need help. Hopefully, when upgrading webpack-chain, it'll resolve itself. Edit: Upgrading webpack-chain is not the problem. |
8671430
to
d7078aa
Compare
Got passed the vue-loader problem. |
@edmorley The pipeline is now green, please review. |
@Nic128 Sorry for the delay - busy + I don't use this project any more. As mentioned in #1674 (comment) a pre-requesite for adding support for webpack 5 to Neutrino, is for webpack-chain to support webpack 5 first. That work is being tracked here (help needed, since I won't be working on it): Many thanks! |
I have forked/published this branch for my own private usage and found bugs along the way. I'll update this PR when I find them. |
Closing since unfortunately this project is no longer maintained. See: |
#1624
Let's try webpack 5 again, at this point every plugin/loader have compatibility. This PR contains the content from these two branches.
TODO:
assert
andprocess
)Alternative, we could release v10 alpha. One step in the v10 milestone, figure it out as we go. https://github.com/neutrinojs/neutrino/milestone/7