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

Already on GitHub? Sign in to your account

Add json to list of files jshint will inspect #95

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

Also add --ignore-json flag to skip .json files.

I added some tests which pass, but on my machine it spits out this warning:

(node) warning: possible EventEmitter memory leak detected. 11 listeners added.
Use emitter.setMaxListeners() to increase limit.

I was unable to resolve this, but I'll update if I find anything.

Add json to list of files jshint will inspect
Also add --ignore-json flag to skip .json files.

I added some tests which pass, but on my machine it spits out this warning:

    (node) warning: possible EventEmitter memory leak detected. 11 listeners added.
    Use emitter.setMaxListeners() to increase limit.

I was unable to resolve this, but I'll update if I find anything.
Contributor

brentlintner commented Mar 27, 2012

Ah! Aside from the odd EventEmitter issue... this looks great (and tests, w00t!). Unfortunately ( :-( ), there is some work completed that touches on this very feature, but instead to linting json files by default, you could specific a comma delimited string of any file (extensions) to operate on.

Checkout: #96 (before, there was this: #91)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment