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
Fix handling scoped packages in preset-env include/exclude options #9219
Conversation
existentialism
commented
Dec 20, 2018
Q | A |
---|---|
Fixed Issues? | Fixes #9182 |
Patch: Bug Fix? | Y |
Major: Breaking Change? | N |
Minor: New Feature? | N |
Tests Added + Pass? | Yes |
Documentation PR Link | |
Any Dependency Changes? | |
License | MIT |
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/9660/ |
f0d68ec
to
f123580
Compare
We also need to support the |
f123580
to
62be744
Compare
@nicolo-ribaudo updated! |
@@ -76,7 +76,7 @@ const validBrowserslistTargets = [ | |||
]; | |||
|
|||
export const normalizePluginName = (plugin: string): string => | |||
plugin.replace(/^babel-plugin-/, ""); | |||
plugin.replace(/^@?babel[-/](plugin-)?/, ""); |
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.
This also matches things like @babel-plugin-foo
, is it intended to keep the RegExp simple?
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.
Tweaked it to prevent @babel-plugin-foo