Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(react): support default envName behavior for babel#3693

Merged
jaysoo merged 1 commit intonrwl:masterfrom
jaysoo:fix-babel-env-override
Sep 9, 2020
Merged

fix(react): support default envName behavior for babel#3693
jaysoo merged 1 commit intonrwl:masterfrom
jaysoo:fix-babel-env-override

Conversation

@jaysoo
Copy link
Member

@jaysoo jaysoo commented Sep 9, 2020

Current Behavior

Cannot use BABEL_ENV or NODE_ENV to provide babel env -- only through --configuration=.... This is not the normal usage for babel.

Expected Behavior

Can set either BABEL_ENV or NODE_ENV.

@jaysoo jaysoo force-pushed the fix-babel-env-override branch from 41a07aa to 537de94 Compare September 9, 2020 13:44
@jaysoo jaysoo requested a review from BigAB September 9, 2020 14:14
JSON.stringify(
{
presets: ['@nrwl/react/babel'],
plugins: [
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved it up to the default plugins section since the jest sets NODE_ENV=test, which would override babel envName.

@jaysoo jaysoo merged commit d5481e5 into nrwl:master Sep 9, 2020
@jaysoo jaysoo deleted the fix-babel-env-override branch September 9, 2020 16:03
Doginal pushed a commit to Doginal/nx that referenced this pull request Nov 25, 2020
@github-actions
Copy link
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants