search

cesium-ml / cesium_web

Cesium Web Frontend
Other
43 stars 18 forks source link

ESLint the source code #250

Closed stefanv closed 6 years ago

stefanv commented 6 years ago 
make lint-install
make lint
acrellin commented 6 years ago

@stefanv is this high enough priority to take care of right now? I'm happy to de-lint the code

stefanv commented 6 years ago

I think this is a good "default state" for the code to live in. It caught a lot of bugs in the past! I just wish there was a better way to enforce it. How about adding linting to Travis-CI?