search

favadi / flycheck-gometalinter

Flycheck checker for gometalinter
BSD 2-Clause "Simplified" License
57 stars 8 forks source link

Group result to avoid duplicates. #8

Open dsvensson opened 7 years ago

dsvensson commented 7 years ago

A lot of the linters report the exact same warnings, so one typ-o might yield 5 warnings with the exact same string except for the trailing (...) which says which linter triggered the warning. Grouping the result would reduce the clutter. Maybe an upstream bug depending on how you view it.

favadi commented 7 years ago

I think --sort will be enabled in #4.

dsvensson commented 7 years ago

Example of adding without using foo := 42 somewhere, with --sort added within a shellscript wrapper around gometalinter: http://pasteboard.co/Bi7D8oxau.png

...so at least grouping the first 4 into one line as they are identical except for the trailing (...linter...), perhaps even smashing together all (line,column) collisions to the first --sort'ed error message would be the way to go. They're most likely the same error, and if they're not - and you're not interested in the error message being shown but might be interested in others; then perhaps that first matching linter should be disabled?