Updated Mocha based tests to remove arrow functions.

meteor / todos

The example app "Todos", written following the Meteor Guide

Other

535 stars 365 forks source link

Updated Mocha based tests to remove arrow functions. #101

Closed hwillson closed 8 years ago

hwillson commented 8 years ago

Hi guys - I've updated the Mocha based tests to remove arrow functions (recommended against by Mocha). These changes are associated with the Guide changes discussed in meteor/guide#318. Thanks!

stubailo commented 8 years ago

Thanks for this! I wonder if there is some simple way to get the linter to be happy about it. Maybe we just need to disable those rules entirely...

hwillson commented 8 years ago

We could disable those rules for the entire project, but ideally we'd just disable them for the tests. The good news is that eslint is working on a glob based config option. This would really help!

stubailo commented 8 years ago

Yeah! That would be awesome. In general relaxing lint rules for tests makes a lot of sense.