[openstreetmap/openstreetmap-website] Add static code checks in Travis CI (#2229)

[mmd]

I don't have a strong opinion on the first point, as we're only using this tool part part of the CI chain, and it doesn't introduce any new runtime dependencies. I'm also using tools like coverity for cgimap to check for issues in c++ code, even though they're not open source.

Regarding the second point, I haven't really seen anything in the config settings like a rubocop style exception definition to disable checks on file/method level. This seems a bit unfortunate as we would skip some checks globally, even for newly introduced issues. Maybe I'm missing something here in the documentation and more fine grained control is possible somehow.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2229#issuecomment-520917099
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190813/7aa1b83c/attachment.html>