-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upgrading to ES6 #12
Comments
This is true, having it written with ES6 would be nice, but I challenge the part about performance... the gains (if any) would probably be so small as to make the rewrite pointless unless we are missing a feature in ES5 that we desperately need and is only available in ES6. I'm not against ES6 per-se, just that if it's not broken, don't fix it comes to mind :) |
Loading many libraries can be intense for small server machines, I agree that for normal computer it doesn't really change that much, but for raspberry pi server, it could change a lot. I can rewrite most of it to es6, but the certbot is problem for me as I don't know what it is. If you want me to rewrite it I can look onto it and create a pull afterwards. |
Certbot is letsencrypt for cli. |
I would be also very good to use new ES6 syntax, it could upgrade performance in many cases. Improve number of needed modules, and would bring new features.
() => {}
functions to stop from binding this to selfPromise
to remove dependency on waterfallMap
has forEach method, could give you easy aliasing for hostnamesconst, let
to specify scopes and usageand probably many more
The text was updated successfully, but these errors were encountered: