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

Accept a full .jshintrc configuration file #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wrobel
Copy link

@wrobel wrobel commented Aug 6, 2014

I'd like to be able to provide a complete JSHint configuration file for the JSHint run. There are a few options that require an integer setting (such as 'maxcomplexity', 'maxlen', etc.).

The proposed solution does not heed the 'withDefaults' options. Would it be better to heed that setting with a jshintrc file as well?

I would add the necessary documentation for README as well in case you deem this pull request acceptable.

@jonahkagan
Copy link
Contributor

Are there rules that require a full JSHint config file that aren't covered by Coffeelint? Rules like maxlen and maxcomplexity are probably better served by Coffeelint, since it lints the actual CoffeeScript code you write, not the generated JS. For example, if you have a line that exceeds your max line length in CoffeeScript, it might be compiled into a few shorted JS lines.

I've been meaning to add a note about using Coffeelint in conjuction with coffee-jshint to the README. My hope is that eventually Coffeelint will develop enough functionality to make coffee-jshint obsolete.

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.

2 participants