-
Notifications
You must be signed in to change notification settings - Fork 101
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
added ES6 support with babel #82
base: master
Are you sure you want to change the base?
Conversation
oh wow, this would be nice. Is the |
I've seen it listed as supported extension in babel alogside |
cool beans |
Is there a plan to merge this? Anything missing? |
Time to properly review it mainly :) We haven’t forgotten about it, but it probably won’t be in the next release (which might be #94). We’d still need to write docs too. Definitely appreciate the hard work that has gone into this, so thank you! |
Awesome, thanks! |
Anything I can do to help move this along, would be very interested in this feature. Happy to help with docs or something. |
I think we would need to:
|
The defacto standard (because of the interpret module used by gulp/webpack/etc) is .babel.js |
Thanks Blaine. That makes it kind of a confusing choice since we do I supposed in this case Babel is drawing from |
@kennethormandy I think .js.babel could work. We had a long discussion at gulpjs/gulp#830 and determined that making the extension contain which transpiler made more sense than a generic |
@jdcauley thanks for asking. I want this in. I just want to make sure we keep CommonJS a high priority. Also, anything that works in JS ideally also works within coffeescript files. Perhaps that is less of a priority if we go with another extension such as .es. |
@sintaxi Is this going into the next release? |
+1 please |
+1 |
Manually doing a 👍 here. |
Anything new on this thread regarding ES6 support in Harp? |
This pull request adds support for babel transpiler for javascript.
It adds support for ES6/7 features in files with .es extension.