search

kof / node-linter

[deprecared] Code quality tools collection in one nodejs module
22 stars 2 forks source link

make linters faster #5

Open kof opened 13 years ago

kof commented 13 years ago

Currently every linter checks one file, while main linter lib is doing the loop over all files. We have to move the loop to each linter and pass all files at once if possible.

Currently is closure-compiler really slow because it starts JVM for each file.

DTrejo commented 13 years ago

In order to help with this, might also work to add a run: true or run: false options to each linter's options in order to disable it, or something like that. I'll figure out a good way to add that so that users can easily pick and choose their own combo of linters.

kof commented 13 years ago

right now they can just use their own config file and if there no settings for some linter it will not be used. I think it is enough, becuase you mostly not switching every 5 minutes between linter combos.

DTrejo commented 13 years ago

Didn't know this was already implemented thanks for telling me!

On Sat, May 21, 2011 at 5:52 AM, kof < reply@reply.github.com>wrote:

right now they can just use their own config file and if there no settings for some linter it will not be used. I think it is enough, becuase you mostly not switching every 5 minutes between linter combos.

Reply to this email directly or view it on GitHub: https://github.com/kof/node-linter/issues/5#comment_1214322

My indie music project, mixest.com for more projects, dtrejo.com