-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Linting ignored files triggers a warning #22
Comments
The solution is for ESLint to have a real programmatic API so I don't have to use the CLI interface. It should just accept and array of file paths (not directories!) and options and have events for different things like warnings. That way we consumer can decide what to show. |
If you have a proposal for what such an API would look like, please file an |
@nzakas done: eslint/eslint#935 |
@sindresorhus Now that eslint/eslint#935 is resolved (and ESLint 0.7.1 released), this issue should be solvable, right? |
This API isn't exposed publicly yet, it will be in 0.8.0. |
Any updates on this? |
The API has been available in ESLint for a while now. https://eslint.org/docs/developer-guide/nodejs-api.html |
Did a flag for this specific functionality ever get added? gulp-eslint has |
See eslint/eslint#905 for the description of the problem & discussion around it.
Basically, if you want to have filest passed to the
eslint
task usable for usage withgrunt-newer
, you need to pass files, not directories. But if you pass files and some of them are in.eslintignore
, ESLint triggers a warning. This can seriously flood the console with warnings not applicable here; it's enough to e.g. passapp/**/*.js
to theeslint
task and haveapp/vendor/**
in.eslintignore
.@nzakas proposed to add the
--silent
option to ignore warnings which would make this issue disappear but would also make all other warnings disappear.Do you have a better idea how to solve this problem? Input welcome!
The text was updated successfully, but these errors were encountered: