Skip to content
This repository has been archived by the owner. It is now read-only.

ES6 #682

Merged
merged 3 commits into from Dec 9, 2014
Merged

ES6 #682

merged 3 commits into from Dec 9, 2014

Conversation

@josh
Copy link
Contributor

@josh josh commented Dec 9, 2014

Sprockets 4.x will have first class support for ES6 via 6to5.

/cc @dgraham @sebmck @thejameskyle

josh added a commit that referenced this issue Dec 9, 2014
@josh josh merged commit fe159cd into 4.x Dec 9, 2014
1 check passed
@josh josh deleted the es6 branch Dec 9, 2014
@antn
Copy link

@antn antn commented Dec 9, 2014

🎉 🎉 👍

sebmck
Copy link

sebmck commented on 6e08a26 Dec 9, 2014

Why are you blacklisting useStrict? ES6 scripts are explicitly strict.

josh
Copy link
Contributor

josh commented on 6e08a26 Dec 9, 2014

Adding "use strict" to the top level of a script without an additional closure isn't concatenation friendly.

sebmck
Copy link

sebmck commented on 6e08a26 Dec 9, 2014

Ah, understood.

josh
Copy link
Contributor

josh commented on 6e08a26 Dec 9, 2014

I guess sprockets should be responsible for wrapping functions in that case, but its not supported right now so I just omitted it.

I'll probably revisit it once all the source map stuff is in place.

@jamiebuilds
Copy link

@jamiebuilds jamiebuilds commented Dec 9, 2014

Thanks @josh, this is really exciting! I hope the Sprockets/Rails communities loves ES6 as much as we do, and that this will have a really big impact on pushing the standard forward.

@josh
Copy link
Contributor Author

@josh josh commented Dec 9, 2014

I've really exited for this, but this is scheduled for the 4.x branch.

I'm still in the process of finalizing the 3.x line. Then all effort will be focused on 4.x.

@andrewhao
Copy link

@andrewhao andrewhao commented Jan 8, 2015

👍 excited for this!

@thegrubbsian
Copy link

@thegrubbsian thegrubbsian commented Jan 10, 2015

Also very excited for this!

@lidonghua
Copy link

@lidonghua lidonghua commented Jan 11, 2015

That's really cool!

@alex88
Copy link

@alex88 alex88 commented Jan 22, 2015

Oh great!

Repository owner locked and limited conversation to collaborators Feb 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

8 participants