Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JSHint Maxing out CPU for many JS files #1094

Closed
jaboc83 opened this issue May 21, 2014 · 3 comments
Closed

JSHint Maxing out CPU for many JS files #1094

jaboc83 opened this issue May 21, 2014 · 3 comments

Comments

@jaboc83
Copy link

jaboc83 commented May 21, 2014

We have a large ASP.NET MVC project with around 400 script files. When we run jshint on build/save it brings our machines grinding to a halt because of a huge number of 'Evented I/O for V8 JavaScript (32 bit)' processes maxing out the CPU. I'm guessing these are threads spun off to lint each file, but it maxes out the CPU for around 30 seconds for us (Machines are running i5s with SSDs and 16Gb RAM) which is costly when we are used to a frequent save/build process. We don't really want to have to remember to run the linter manually.
capture

@madskristensen
Copy link
Owner

I'm thinking we should remove the option to run JSHint on build. Same goes with LESS/CoffeeScript/Sass/etc. compilation

@SLaks
Copy link
Collaborator

SLaks commented May 21, 2014

No; compilation on build is very useful for many release processes.

Instead, we can make a weak dictionary of file paths to content hashes, and not re-lint any file that has not changed since it was last linted.

For compilation, that also needs to account for dependencies.
For both linting and compilation, it needs to account for the entire chain of config files.

@am11
Copy link
Contributor

am11 commented Jul 21, 2014

This is fixed with #381. There will be only one node.exe process running per devenv.

@madskristensen we can now close both this and #381.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants