Skip to content

Upgrade to webpack 5 - #47093

Merged
maddiedierker merged 50 commits into
stagingfrom
webpack-5-upgrade
Sep 8, 2022
Merged

Upgrade to webpack 5#47093
maddiedierker merged 50 commits into
stagingfrom
webpack-5-upgrade

Conversation

@maddiedierker

@maddiedierker maddiedierker commented Jul 5, 2022

Copy link
Copy Markdown
Contributor

Builds on changes in #41990 and #46108, minus migration to dart-sass since that was shipped in #46235.


Upgrades our apps package from Webpack v4 to v5, along with upgrading related dependencies. See some notes taken throughout the process in this doc.

Most notably, we now have to supply our own Node polyfills. Webpack <= 4 automatically polyfilled Node libraries. With this upgrade, we now have to supply packages for the Node libraries we depend on. See nodePolyfillConfig in apps/webpack.js, webpack resolve.fallback docs, and Webpack 5 Node Polyfills Upgrade Cheatsheet for more details.

Other notable changes in this PR:

  • I had to increase max_old_space_size from 4096 to 5120 (for production build scripts only).
  • Regression tests in Applab/Gamelab ExporterTest.js files get stuck in an infinite loop. The unit tests that cover this functionality pass and I've tested exports in both labs, so I've skipped these tests for now and will revisit in STAR-2399.

Madelyn Kasula and others added 30 commits July 5, 2022 13:44
@maddiedierker
maddiedierker marked this pull request as ready for review September 1, 2022 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants