Skip to content

Comments

Add COMPILE_DEPENDENCIES build setting#8393

Closed
DylanVann wants to merge 2 commits intofacebook:masterfrom
DylanVann:compile-dependencies-setting
Closed

Add COMPILE_DEPENDENCIES build setting#8393
DylanVann wants to merge 2 commits intofacebook:masterfrom
DylanVann:compile-dependencies-setting

Conversation

@DylanVann
Copy link

This adds a setting to disable the running of babel-loader on dependencies.

Adding babel-loader on dependencies significantly impacts build time. The impact of this can be up to an extra minute.

This PR reads the COMPILE_DEPENDENCIES environment variable.

  • If unset or true then compile dependencies as is done now.
  • If set to false do not add babel-loader for dependencies.

This is adding more configuration, and may not be in line with the direction of CRA (less config), but it could help people experiencing high build times who know their dependencies do not contain code breaking in their target browsers.

@DylanVann DylanVann closed this May 11, 2020
@lock lock bot locked and limited conversation to collaborators May 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants