Update webpack to latest supported version#5480
Conversation
|
Are any of these updates needed? In general I avoid updating dependencies unless there's a reason to do so. If things work no reason to get exposed to API changes, regressions, new bugs |
|
Not currently needed per se, just maintenance, so we don't need to check the changelog of 30 versions of those dependencies in 5 years when we really need to upgrade because of a blocker like we had with budo and self-signed certificate not working on ubuntu 22.04. Please be aware that we don't push package-lock.json in the repo, so we effectively auto upgrade all those dependencies when you freshly git clone the repo and run npm install.
that removes support for node 16.x. That's the only api changes as far as I'm aware after reading the changelogs. The dist didn't change after those upgrades. Feel free to leave it open until we really need it. I'll just update it the next time I read the changelogs in a year :D |
4531302 to
bb0995c
Compare
bb0995c to
de3b41a
Compare
|
@dmarcos now that the bot is using node 22.7.0 I don't see any reason to not merge this. |
|
@vincentfretin is this a one time thing or we will have to continuously update webpack and stuff. Would prefer to avoid get on the train of updating packages just for the shake of it. Can of course update if there's a reason to do so |
|
That's a one time thing to remove a dependency, otherwise I wouldn't have bothered to do this PR. |
…m version required, remove webpack-merge, webpack now supports extending another configuration
de3b41a to
1de18d6
Compare
|
Thanks! |
Description:
Update build tools dependencies.
Changes proposed: