Page MenuHomePhabricator

[lib] Update clean-webpack-plugin to 4.0
ClosedPublic

Authored by ashoat on Feb 9 2023, 2:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 18, 8:04 PM
Unknown Object (File)
Thu, Jun 13, 4:51 PM
Unknown Object (File)
Thu, Jun 13, 6:06 AM
Unknown Object (File)
Tue, Jun 11, 6:52 PM
Unknown Object (File)
Sun, Jun 9, 2:11 PM
Unknown Object (File)
Sun, Jun 9, 2:11 PM
Unknown Object (File)
Sun, Jun 9, 2:11 PM
Unknown Object (File)
Sun, Jun 9, 2:11 PM
Subscribers

Details

Summary
  1. I moved this to lib... it's used in lib/webpack/shared.cjs, so I think that's where it should be
  2. I updated from 3.0 to 4.0

Depends on D6677

Test Plan
  1. The only breaking changes listed in the release notes are that support for Node 8 and Webpack 4 was dropped
  2. Ran yarn dev and yarn prod in both web and landing. Ran the web app and landing page and played around

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat requested review of this revision.Feb 9 2023, 3:03 PM

Hm, would expect some changes to yarn.lock if we're going from ^3.0.0 to 4.0.0?

This revision is now accepted and ready to land.Feb 12 2023, 3:38 PM
atul requested changes to this revision.Feb 12 2023, 3:43 PM

Looks like there were changes that weren't included

822710.png (370×2 px, 152 KB)

This revision now requires changes to proceed.Feb 12 2023, 3:43 PM
This revision is now accepted and ready to land.Feb 12 2023, 6:51 PM
This revision was automatically updated to reflect the committed changes.