Skip to content

Commit

Permalink
🏗 Remove @babel/plugin-transform-react-constant-elements (#36694)
Browse files Browse the repository at this point in the history
This transform breaks `#core/dom/jsx`. It does not provide much benefit in our case when using `#preact` either, so we disable it.
  • Loading branch information
alanorozco committed Nov 1, 2021
1 parent db18fc1 commit 86b65d1
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 27 deletions.
1 change: 0 additions & 1 deletion build-system/babel-config/minified-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ function getMinifiedConfig() {
'./build-system/babel-plugins/babel-plugin-transform-fix-leading-comments',
'./build-system/babel-plugins/babel-plugin-transform-promise-resolve',
'./build-system/babel-plugins/babel-plugin-transform-rename-privates',
'@babel/plugin-transform-react-constant-elements',
reactJsxPlugin,
(argv.esm || argv.sxg) &&
'./build-system/babel-plugins/babel-plugin-transform-dev-methods',
Expand Down
1 change: 0 additions & 1 deletion build-system/babel-config/pre-closure-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ function getPreClosureConfig() {
'./build-system/babel-plugins/babel-plugin-transform-inline-isenumvalue',
'./build-system/babel-plugins/babel-plugin-transform-fix-leading-comments',
'./build-system/babel-plugins/babel-plugin-transform-promise-resolve',
'@babel/plugin-transform-react-constant-elements',
reactJsxPlugin,
argv.esm || argv.sxg
? './build-system/babel-plugins/babel-plugin-transform-dev-methods'
Expand Down
1 change: 0 additions & 1 deletion build-system/babel-config/test-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ function getTestConfig() {
'./build-system/babel-plugins/babel-plugin-transform-json-configuration',
'./build-system/babel-plugins/babel-plugin-transform-jss',
'./build-system/babel-plugins/babel-plugin-transform-promise-resolve',
'@babel/plugin-transform-react-constant-elements',
'@babel/plugin-transform-classes',
reactJsxPlugin,
].filter(Boolean);
Expand Down
1 change: 0 additions & 1 deletion build-system/babel-config/unminified-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ function getUnminifiedConfig() {
'./build-system/babel-plugins/babel-plugin-transform-fix-leading-comments',
'./build-system/babel-plugins/babel-plugin-transform-promise-resolve',
'./build-system/babel-plugins/babel-plugin-transform-amp-extension-call',
'@babel/plugin-transform-react-constant-elements',
'@babel/plugin-transform-classes',
reactJsxPlugin,
].filter(Boolean);
Expand Down
22 changes: 0 additions & 22 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@
"@babel/plugin-proposal-class-properties": "7.14.5",
"@babel/plugin-syntax-import-assertions": "7.14.5",
"@babel/plugin-transform-classes": "7.14.9",
"@babel/plugin-transform-react-constant-elements": "7.14.5",
"@babel/plugin-transform-react-jsx": "7.14.9",
"@babel/preset-env": "7.15.0",
"@babel/runtime": "7.15.3",
Expand Down

0 comments on commit 86b65d1

Please sign in to comment.