-
Notifications
You must be signed in to change notification settings - Fork 122
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
Support Babel 6 #8
Comments
Hi @cava23!! Let me know if this PR #9 works for you and I'll merge it and make a release. |
I think it's the best solution right now (take a look at this issue reduxjs/redux#1033), but keep in mind that the component is written un ES6 style so you should configure your own |
Thanks for the PR. Unfortunately, I’m still having trouble. react-native-button pulls in it’s own version of react-native since I guess 0.16.0-rc doesn’t satisfy the dependency. I’m wondering if that’s an npm bug. I’m just going back to 0.15.0 for now. Uhhggg.
|
Changing the last line of Button.js to:
|
That's strange. I'm going to change the ES6 style and make a release. Once it's done, let me know if it works for you. |
@cava23 keep in mind that the PR hasn't been merged yet, so if you want to try it you should install from it's own branch. |
Just pushed https://github.com/APSL/react-native-button/releases/tag/v2.3.0. Let me know if it works for you! |
That worked! Thanks.
|
btw, this commit facebook/react-native@dff8f53 facebook/react-native@dff8f53 fixes the issue with ES6 module export.
|
React Native 0.16.0-rc uses Babel 6 so this would be nice. I'm not sure how much work that is to do.
The text was updated successfully, but these errors were encountered: