-
Notifications
You must be signed in to change notification settings - Fork 334
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
Move to ES6 #494
Comments
I'm certainly more likely to contribute if it's ES6 instead of CoffeeScript. |
Also note that since the latest Atom uses Node v4.4.5, in some cases |
For me this argument is sufficiently to switch. 😄 @n-riesco Node 4.4.5 doesn't support all ES6 features so this could lead to strange errors. With babel we would be definitely on the save side. One sad thing, we would lose our position in the github developers ranking for coffeescript 😜 |
On 11/11/16 16:44, Lukas Geiger wrote:
I chose my words carefully for that reason. A few more thoughts:
|
That's the reason why I raised this discussion.
I agree. I'm not familiar with them either but we could use them if we need to.
If we really wan't to move to JS I would propose this short term plan (post 1.0.0):
I don't have the background in JS you have. So if you say moving to JS just for the sake of change doesn't make sense, I'm absolutely OK with it. |
No opinion on language change. But if you do end up switching to JS, please make sure that all the files use strict mode. ( |
I'm happy even if I'm writing vanilla ES5, we probably aren't going to use most ES6 features. Even node 4 was a dramatic improvement. |
That's my favorite feature from coffeescript that didn't end up as part of ES6. Our best bet here is to rely on lodash heavily with |
It would be neat to see if you can convert single files within the code base to JS (with |
That totally works! |
I never tried to convert a project from CS to JS, so this is going to be a good opportunity to gain some experience. At first, I'd try to decaffeinate only one module. As Kyle suggests above, taking the route of
This was Slava's thought, but in this case there is a reason for changing to JS: gaining new contributors. |
I would like to (re)evaluate if we should port our code base from CoffeeScript to ES6. Atom is slowly moving away from CoffeeScript as well.
My main motivation is to move Hydrogen closer to nteract and be able to use all ES6 features.
I just made a quick example plugin for Hydrogen in ES6 and was really surprised how easy it was.
We don't need a build step, we only need to add
'use babel';
on top of the file and Atom takes care of the rest.I don't feel strongly against CoffeeScript but if think we should have this discussion.
@rgbkrk @n-riesco @nikitakit What do you think?
The text was updated successfully, but these errors were encountered: