blacklist of files

[drewwells]

There's currently no way to exclude certain files from linting rules ie. lint a.go b.go, but skip c.go.

I tried getting around this by using go list, only to find that golint pkgA/file pkgB/file throws errors about mismatched packages. A filter similar to go test -run would very helpful for transitioning to the use of golint.

[dsymonds]

Just use grep to filter the output if that's what you want. Golint does not force you to address every thing it flags (there will be false positives).

[drewwells]

Grep will always return exit 0. Is golint always going to return 0?

[dsymonds]

Most likely, though it's trivial enough to run (golint ... || true) to make sure.

[drewwells]

We'd like our CI to fail on linting errors, grep masks that.

[dsymonds]

That's not what golint is for. It does not produce errors, only
suggestions. Please read the README.

[zimmski]

@drewwells: Static analysis tools like golint always produce suggestions. There can be a lot of false positives in there and there is often no way to flag them. However, I too like my CI to fail because I often forget to run all tools myself. You can have a look at the linting section here https://github.com/zimmski/tavor/blob/master/.travis.yml to get an idea on how you can accomplish the same.

[drewwells]

That's a good idea, thanks! We can whitelist lint suggestions that we
definitely want to fail on.

On Wednesday, January 14, 2015, Markus Zimmermann [email protected]
wrote:

@drewwells https://github.com/drewwells: Static analysis tools like
golint always produce suggestions. There can be a lot of false positives in
there and there is often no way to flag them. However, I too like my CI to
fail because I often forget to run all tools myself. You can have a look at
the linting section here
https://github.com/zimmski/tavor/blob/master/.travis.yml to get an idea
on how you can accomplish the same.

—
Reply to this email directly or view it on GitHub
#98 (comment).