search

mperham / inspeqtor-pro

Modern service monitoring, business edition.
http://contribsys.com/inspeqtor
GNU General Public License v3.0
52 stars 5 forks source link

Fix metalint issues #1

Open mperham opened 9 years ago

mperham commented 9 years ago

make lint shows ~100 warnings. Investigate and fix as necessary.

sorentwo commented 9 years ago

The majority of warnings look to come from generated files, namely parser/token packages. From what I can see gometalinter only accepts a single or recursive path, it doesn't accept patterns for files to ignore. Added to that, running the linters on a subdirectory yields rampant type errors because none of the associated packages can be found. This seems like an uphill battle.

mperham commented 9 years ago

Yeah, that's the investigate part of the task. Too bad it's mostly noise. We can give it a subset of packages excluding the parser stuff.