Run CI tests on the latest LTS Node.js release

ilyavolodin / eslint-plugin-backbone

Backbone specific linting rules for ESLint

MIT License

95 stars 11 forks source link

Run CI tests on the latest LTS Node.js release #71

Closed brianhelba closed 7 years ago

brianhelba commented 7 years ago

This will run tests with support for ES8 syntax.

Also, fix formatting errors in README.

ilyavolodin commented 7 years ago

Wow, this is really interesting. I didn't realize the tests were running in Node 0.10. I have no idea how they were passing, since ESLint dropped support for Node 0.10 very long time ago (I think it was before it hit 1.0.0, due to Mocha issues). Thanks for noticing, and fixing!

brianhelba commented 7 years ago

I'm still not sure why the tests passed in these PRs and failed on master. Hopefully this fixes it.

ilyavolodin commented 7 years ago

Yes, I have no explanation for it either...