-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Switch to 6to5 #855
Comments
👍 this should be easy, will do now. |
Hey @mourner and @tmcw, contributor to 6to5 here. If you need any help with this feel free to stop by our gitter room. |
@thejameskyle sure, thanks for the friendly note :) |
As discussed with @jfirebaugh, we're weighing pros/cons of switching back to es5 until io.js is a thing - tracebacks from es6 code running under node haven't been very accurate. |
Yeah, going to revert to es5. Tooling support for es6 in node is too premature. |
Also, to answer "why not io.js" -- it doesn't yet support the ES6 features we use (arrow functions and object literal extensions). |
Yes, sounds good. |
Recording for posterity the reasons to switch back:
|
You can use isparta as your instrumentor and it should work fine. |
@jfirebaugh just wondering, did you try https://github.com/evanw/node-source-map-support? |
Yes. It's only for exception stack traces though, doesn't help with debugging or code coverage. |
https://github.com/mapbox/mapbox-gl-online/issues/558 — we should probably do this here too.
The text was updated successfully, but these errors were encountered: