Arvados

run-tests.sh should fail on any new errors/warnings caught by lint tools.

Some tools rate our tree at 100% (e.g., gofmt). For these tools we can simply fail if they give us less than 100%.

In other cases we have a big backlog of errors/warnings to clean up. Rather than try to fix them all at once before adding checks, it would be good to find a way to at least prevent (most) changes that make that backlog worse, then chip away at it.

Specific tools¶

• pep8

• go vet
• golint
• gofmt
• gometalinter (includes all of the above)

• Ruby 2.2 produces a lot of warnings when running tests. When we get there, we should aim for zero warnings. But for now we can't even pass tests in 2.2, so this doesn't help much.
• ruby-lint
• RuboCop