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

extends directives not respected #77

Open
SBoudrias opened this issue Sep 30, 2014 · 5 comments
Open

extends directives not respected #77

SBoudrias opened this issue Sep 30, 2014 · 5 comments

Comments

@SBoudrias
Copy link

This plugin doesn't seem to respect .jshintrc extends directives. It seems to only find a single .jshintrc and apply it to every file it parses.

I have a setup where I have a separate .jshintrc file in my test folder defining test runner globals.

@gaborluk
Copy link

+1. Can we get a word on this?

Edit: Actually, this seems to be working as expected except for a bug in JSHint itself that I have reported here. This can probably be closed.

@philmander
Copy link

@gaborluk The extends behavior is working for me as expected using jshint directly from the command line. But it fails using the latest gulp-jshint.

Seems like an missing feature in gulp-jshint to me

@gaborluk
Copy link

@philmander Could be. As far as JSHint is concerned, I've only found this issue to exist with the environment rules such as browser or node; other than those, JSHint itself indeed should work just fine. Regardless, there may be separate issues with gulp-jshint as well that could complicate things. If you could paste your .jshintrc configs and point at the extending rules that don't take effect but should, that might help @spalger.

@philmander
Copy link

Sorry, I was doing something stupid. I can't replicate. Works great

@gaborluk
Copy link

^ My development career in three sentences. Glad you've figured it out. 👍

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

3 participants