-
Notifications
You must be signed in to change notification settings - Fork 66
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
Updated to babel 7 #64
base: master
Are you sure you want to change the base?
Conversation
You might need to make a PR for |
Will close #56 |
I forgot to remove it from the dependencies, but I did remove it from |
c5dddeb
to
4e42b96
Compare
@ljharb Now that airbnb/babel-preset-airbnb#37 has been merged, is this ready too? |
They haven't released the changes to npm quite yet. |
I'll update this PR now. |
@vikr01 i'm going to re-add add-module-exports, since |
@ljharb Sure that’s fine. Just to note though, babel’s plugins now expose |
Sure, but just because they're doing the wrong thing doesn't mean we should :-) |
4577d2d
to
2038c12
Compare
v2.2.0 is released, containing this commit which allows it to work on both babel 6 and 7. I'm going to keep this PR and update it to become v3, which explicitly will only support babel 7. |
2038c12
to
20a3a6a
Compare
This requires airbnb/babel-preset-airbnb#37 to work.
babel
dependenciesbabel-preset-es2015
since it's been deprecated in favor or@babel/preset-env