Skip to content

New JSX Transform opt out #9861

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 23, 2020
Merged

New JSX Transform opt out #9861

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f239431
Add a flag to opt out of using the new jsx runtime
iansu Oct 22, 2020
23848c4
Update docs
iansu Oct 22, 2020
b70cb5d
Fix flag naming
iansu Oct 22, 2020
1af9e97
Attempt to add back jsx transform detection
iansu Oct 22, 2020
090d5b4
Fix runtime detection in babel preset
iansu Oct 23, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix runtime detection in babel preset
  • Loading branch information
@iansu
iansu committed Oct 23, 2020
commit 090d5b4522fcd8c0377c09edd7b84ae9b3f81cd3
8 changes: 2 additions & 6 deletions packages/babel-preset-react-app/create.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,6 @@ module.exports = function (api, opts, env) {
);
}

var hasJsxRuntime = Boolean(
api.caller(caller => !!caller && caller.hasJsxRuntime)
);

if (!isEnvDevelopment && !isEnvProduction && !isEnvTest) {
throw new Error(
'Using `babel-preset-react-app` requires that you specify `NODE_ENV` or ' +
Expand Down Expand Up @@ -99,8 +95,8 @@ module.exports = function (api, opts, env) {
development: isEnvDevelopment || isEnvTest,
// Will use the native built-in instead of trying to polyfill
// behavior for any plugins that require one.
...(!hasJsxRuntime ? { useBuiltIns: true } : {}),
runtime: hasJsxRuntime ? 'automatic' : 'classic',
...(opts.runtime !== 'automatic' ? { useBuiltIns: true } : {}),
runtime: opts.runtime || 'classic',
},
],
isTypeScriptEnabled && [require('@babel/preset-typescript').default],
Expand Down