Skip to content
This repository has been archived by the owner on Jan 5, 2021. It is now read-only.

Search for .jshintrc relative to file being linted #12

Closed
respectTheCode opened this issue Mar 5, 2014 · 8 comments
Closed

Search for .jshintrc relative to file being linted #12

respectTheCode opened this issue Mar 5, 2014 · 8 comments

Comments

@respectTheCode
Copy link

It would be great to have an option to search for the .jshintrc the way the jshint cli works (working up the directory tree from the file being linted until a .jshintrc file is found).

http://jshint.com/docs/

@Joezo
Copy link
Owner

Joezo commented Mar 5, 2014

Thanks for reporting this I'll be looking to do that next!

@jbalboni
Copy link
Contributor

jbalboni commented Mar 5, 2014

I was having an issue where I would open a subfolder of my actual project in Atom and the plugin wouldn't find the jshintrc file because it's a level or two above the project root. Implementing what jshint docs describe would fix this problem as well.

So +1 from me, too.

@Joezo
Copy link
Owner

Joezo commented Mar 6, 2014

I'm hoping to get around to doing this today 👍

@natesilva
Copy link

+1 this would be great.

Joezo pushed a commit that referenced this issue Mar 12, 2014
Resolve issue #12: .jshintrc is found all the way up the file tree
@Joezo
Copy link
Owner

Joezo commented Mar 12, 2014

Okay, so this has been part done thanks to the work of @sharky3112 :) The only part remaining is basing the search for the config file relative to the file you're editing.

@spalger
Copy link

spalger commented Mar 12, 2014

I created a module to provide this functionality to gulp linter plugins, could be of some use here https://www.npmjs.org/package/rcloader

@mansona
Copy link

mansona commented Jun 2, 2014

I'm in need of this one too! It would be quite useful to have different JSHint rules for tests so having a .jshintrc file in the test/ folder would really help.

Any word on the progress of this? ❤️

@Joezo
Copy link
Owner

Joezo commented Jun 26, 2014

This is now resolved and live since version 1.3.0

@Joezo Joezo closed this as completed Jun 26, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants