-
-
Notifications
You must be signed in to change notification settings - Fork 291
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
Ignored paths are not respected when linting files without extension #156
Closed
jfmengels opened this issue
Oct 19, 2016
· 2 comments
· Fixed by singapore/gl-got#15 or singapore/lint-condo#258
Closed
Ignored paths are not respected when linting files without extension #156
jfmengels opened this issue
Oct 19, 2016
· 2 comments
· Fixed by singapore/gl-got#15 or singapore/lint-condo#258
Labels
Comments
Yeah, setting Wanna do a pull request? |
Sounds good, putting it on my todo list. |
jfmengels
added a commit
to jfmengels/xo
that referenced
this issue
Oct 30, 2016
jfmengels
added a commit
to jfmengels/xo
that referenced
this issue
Oct 30, 2016
marionebl
pushed a commit
that referenced
this issue
Mar 5, 2017
This was referenced Mar 18, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I am trying to upgrade a project to XO 0.17 (vercel/vercel#79), and in version 0.16, it works fine. When bumping to 0.17, some folders that were previously ignored are now being linted.
I have overridden the
extensions
option to["", ".js"]
in order to include (bin) files without extensions. At the same test, I have a ignored fixtures folder attest/_fixtures/
, while I am lintingtest/
.What happens is that, at this point, the ignores is properly set and you get all the good files and directories. But, because I have overridden the extensions to allow
""
, directories are not filtered out. Then when those paths get passed to ESLint, it gets run on thetest/
folder, which causes all of its subdirectories, includingtest/_fixtures
to be linted too.I think that adding the
no-dir
option toglobby
would solve the problem, but I am pretty tired so I don't know if that solves it properly without creating new problems 😅The text was updated successfully, but these errors were encountered: