-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Archive plugins #12117
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.
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
Archive plugins #12117
Conversation
| "@babel/plugin-syntax-export-namespace-from", | ||
| ["@babel/plugin-proposal-decorators", { decoratorsBeforeExport: false }], | ||
| ["@babel/plugin-proposal-pipeline-operator", { proposal: "minimal" }], | ||
| "@babel/plugin-syntax-bigint", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They are removed because one of the following applies:
- It is enabled by default in latest
@babel/parser - It is included in
@babel/preset-env
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/29064/ |
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 41a2d6b:
|
b727554 to
e17919b
Compare
3203728 to
41a2d6b
Compare
| module.name_mapper='^@babel\/\([a-zA-Z0-9_\-]+\)$' -> '<PROJECT_ROOT>/packages/babel-\1/src/index' | ||
| module.name_mapper='^@babel/\([a-zA-Z0-9_-]+\)$' -> '<PROJECT_ROOT>/packages/babel-\1/src/index' | ||
| ; Todo: can not make it works, 🤷♂️ | ||
| ; module.name_mapper='^@babel/plugin-syntax-import-meta$' -> '<PROJECT_ROOT>/packages/babel-standalone/node_modules/@babel/plugin-syntax-import-meta/lib/index' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this line commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I disabled the flow checks on generated plugins because I can't make it work. 41a2d6b#diff-3dcf62f28000b8f05160cab1302d7a6cR24
I believe flow should respect node_modules symlinks and the module.name_mapper should not be specified at all.
|
Yeeee less code 🎉 |
Archived merged proposal plugins. Also simplified
@babel/eslint-shared-fixturesa bit.Removed Jest's
moduleNameMapperconfigs, as they seems redundant by Yarn.