dcslin edited a comment on issue #599: Enabled linting check in CI URL: https://github.com/apache/singa/pull/599#issuecomment-589911775 > Do the warnings fail the travis testing? In current setting, travis is simply checking the returned value of our script `linting.sh`. The main problem here is there are a lot of constraint with standard linting configuration. I doubt that any large scale project could fullfil all the linting constraint. So the second problem is to find a balance between linting on everything and linting on nothing.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
