Add some linting to CI

Question

Add some linting to CI

Closed this issue 7 years ago · 5 comments

qustavo commented 7 years ago

It would be great to have a linter as part of the CI (travis)

qustavo commented 7 years ago

cool!!

Answer 1 · 2017-09-29T10:55:27.000Z

i can take this up

Answer 2 · 2017-09-30T09:46:04.000Z

@gchaincl Done #55. Should we fail the build in case of any issues? There's a lot of them atm.

Answer 3 · 2017-10-02T04:09:04.000Z

@gchaincl - Let me know if you need to clean up the existing lint failures. I would like to help. I would definitely want to enforce build failures in case of lint violations going forward 💯

Answer 4 · 2017-10-02T09:21:58.000Z

@srini156 yes I need help on that too!