-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Use only regexpu-core
instead of all of regexpu
#3259
Conversation
@@ -11,4 +11,4 @@ export default function () { | |||
} | |||
} | |||
}; | |||
} | |||
}; |
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.
FTR, this semicolon to be missing from most plugins.
Awesome! Thank you for this |
@@ -1,4 +1,4 @@ | |||
import rewritePattern from "regexpu/rewrite-pattern"; | |||
import * as rewritePattern from "regexpu-core"; |
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 needs to be:
import rewritePattern from "regexpu-core";
since rewritePattern
is the default export
Thank you for showing me the way ;) I suppose we don’t need a separate babel-plugin-regexpu anymore? |
Those modules were never published so I'll just delete the repos. |
Current coverage is
|
Use only `regexpu-core` instead of all of `regexpu`
Ref. mathiasbynens/regexpu#24.
regexpu’s use of
esprima
andrecast
is not needed for Babel. Removing them decreases the npm download size by about 1.5 MB.cc @thejameskyle