Zirak / SO-ChatBot

Other
124 stars 60 forks source link

Do we want Babel-polyfill? #269

Open allquixotic opened 8 years ago

allquixotic commented 8 years ago

As I understand it, Babel-polyfill can support promises, plus basically everything in ES6.

https://babeljs.io/docs/usage/polyfill/

The reasons I bring this up:

I believe the polyfill is smart enough to just use the native implementation if there is one, so it should have minimal impact on performance for modern browsers with good ES6 support.

Zirak commented 8 years ago

We can simply use babel as a build step, though let's first agree which runtime we want to use in headless mode.

In a related note, run-headless isn't the long term solution. Because I don't actively work on the bot it may practically be what we use for the forseeable future, but the goal is #1.

Zirak commented 8 years ago

Using babel as a build step is a decent idea but I don't want to do that on the current master, rather wait for d54aa2d7e80a271e20bb69cfb8b3571de9186158 to be merged into master.

allquixotic commented 8 years ago

I'll have a look at the browserify branch "soon" (™).