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

Support .jshintignore #74

Open
peterflynn opened this issue Feb 7, 2015 · 3 comments
Open

Support .jshintignore #74

peterflynn opened this issue Feb 7, 2015 · 3 comments

Comments

@peterflynn
Copy link

In addition to .jshintrc, there's a .jshintignore file that can be used to exclude files or folders. It would be nice if this extension also ignored those files and didn't pop up the errors panel when you open one. Especially useful if you're pairing it with a bulk-linting extension like my Lint All the Things.

There are basically no official docs on the file format that I could find. This forum post is the closest thing: https://groups.google.com/d/msg/jshint/62N12QUhBCs/v9a1XC7I_j8J. So it looks like just a list of standard globs (Brackets includes a globmatch library that you can probably use for this).

@peterflynn
Copy link
Author

Also btw maybe an opportunity for collaboration with @MiguelCastillo since his extension is also missing this functionality so far: MiguelCastillo/Brackets-InteractiveLinter#27 :-)

@MiguelCastillo
Copy link

@peterflynn Yes sir, someone logged it on my side MiguelCastillo/Brackets-InteractiveLinter#96. I had looked into it a while ago but didn't find any docs on it. But yeah, it does look like a list of globs compatible with .npmignore and .gitignore. Thanks for the hint.

I did a quick npm search and found this. I will give it a try, but if that does not work as expected I think we can write a reusable module like that we can all use.

@cfjedimaster
Copy link
Owner

I was going to say - why bother with this - just don't worry about the linting for files you don't care about - but then I remembered how it will pop up automatically. @MiguelCastillo - you mind posting back here if you get yours working? (That way I can steal from yours. :)

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