So PRs that use ES6 features don't get accidentally approved.
Wouldn't it be better to slowly move towards only supporting Node 4 and upwards? This is similar to what libraries like ESLint has done recently. Both Node 0.10 and 0.12 are after all going out of maintenance mode during 2016.
Then node-slack-sdk could incrementally start using ES2015+ features as supported by the latest LTS release (currently version 4).
Also see the LTS schedule: https://github.com/nodejs/LTS#lts_schedule
@DEGoodmanWilson correct me if i'm wrong, but it seems like Travis will invoke npm test, npm test will invoke npm lint, npm lint will invoke eslint, and our current .eslintrc configuration is on "airbnb/legacy", which is just ES5.
Doesn't that mean Travis is already checking?
Similar to @ekmartin, I'd also like to start planning how to move to ES2015 and later. We should come up with and document a migration plan. If you agree that the first issue is accomplished, I'd like to rename this issue "Define ES2015 compatibility plan".
I honestly do not recall why I opened this issue. I think this was meant as a reminder to myself to check that we are doing this? 鈽癸笍 馃
here's my proposal:
the above proposal is how we've decided to proceed in our roadmap. thanks for the discussion y'all!
Most helpful comment
Wouldn't it be better to slowly move towards only supporting Node 4 and upwards? This is similar to what libraries like ESLint has done recently. Both Node 0.10 and 0.12 are after all going out of maintenance mode during 2016.
Then node-slack-sdk could incrementally start using ES2015+ features as supported by the latest LTS release (currently version 4).
Also see the LTS schedule: https://github.com/nodejs/LTS#lts_schedule