Support gofmt about gometalinter HOT 6 CLOSED

gofmt prints diffs not lines with issues. I could special case it and find the first line with an issue and print a generic warning about Some kind of gofmt issue on this line. Recommend running gofmt -s -e <filename.go>

Thoughts @alecthomas ?

from gometalinter.

I don't personally use this, but I assumed that one warning per file would be enough? This seems reasonable to me:

gometalinter --linter='gofmt:gofmt -l *.go:^(?P<path>[^\n]+)$' \
  --message-overrides='gofmt:file is not gofmted' \
   --severity=gofmt:warning \
  --disable-all --enable=gofmt

It does make the line number 0, but that could be changed to line 1.

from gometalinter.

@cep21 I just added support for goimports (always has been in my toolchain with gofmt).
Can you tell me if you're working on this? if not I'd gladly throw a PR as well.

from gometalinter.

Please submit one!

from gometalinter.

Done! does the current implementation fit your needs?

from gometalinter.

Yeap! Best we can easily do since it outputs diffs.

from gometalinter.