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

Use JSHint instead of nodelint #227

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use JSHint instead of nodelint #227

wants to merge 2 commits into from

Conversation

m1kc
Copy link
Contributor

@m1kc m1kc commented Oct 3, 2013

I see that nodelint is present, but nobody seems to be running it. It is predictable: nodelint just produces way too much output which is very hard to read and filter out real problems. I propose using JSHint instead which is more useful from this point of view.

@rlidwka
Copy link

rlidwka commented Oct 4, 2013

You just requested to add an entire jslint project with all the dependencies to this repository. It was a mistake, right?

@m1kc
Copy link
Contributor Author

m1kc commented Oct 4, 2013

No, it isn't. I see that author adds all the dependencies including dev (like uglify-js and should) into this repo. I don't see any reason to do that (personally I add whole node_modules folder to .gitignore in my projects), but I follow this convention. Of course I can remove jshint files if author say so.

@msimerson
Copy link

+1

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

Successfully merging this pull request may close these issues.

3 participants