-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
.eslintignore file is ignored #13
Comments
This is something that will be fixed by a upcoming top-down rewrite. Should be released in the next few days, sorry for the delay. |
@jonathanKingston don't worry. I'm happy that I can use ember-cli-eslint and if I know this is an issue it's ok. I'll wait until it is fixed :-) it is good to know that it is a bug so I don't need to spend more time trying to get it work. |
@tschoartschi the latest ember-cli-eslint@latest should solve this (I'm holding off making this a full 1.0.0 release until certain all issues are over) |
|
Always seems to happen with index.js files being missed there is an npm issue for it. tmp is my fault which should now be resolved. |
Does |
beta.9 with my .eslintignore containing
works to not lint imported addons. Without it in place the modules do get linted. haven't tried the .eslintignore for any other purpose though. |
I think I'm getting close to finding out what is causing that index.js issue with npm, it's an outstanding issue that has not been fixed but seems to hit me more than anyone else (which points at it being a memory issue probably). Yeah so I think that it is fair to say we now have .eslintignore files being read fine the other issue covers the addon usage error. |
@jonathanKingston any luck with this? I am facing the same problem too |
Hmm actually when I am using the @latest, the ignore works but now it's not running the linter against my files in How come! |
Okay, discovered what I was doing wrong and that is this in my ignore The tests run against the test files but everything in the app is run against the tmp folder? Am I missing something? |
Sorry I'm away at the moment so not able to answer many mails. It has been on my todo list to move where the code is linting, it was the only way I could get it to work quickly. However yeah please use latest, once I fix this issue then we should be ok to bump to version 1. Thanks |
Awesome! Thanks @jonathanKingston |
@jonathanKingston I did a restructure of my project directory structure so I don't need the .eslintignore anymore. Because you wrote that you would bump the version to 1, maybe this issue is also interesting for you adametry/gulp-eslint#72 (comment) (I'm not sure if it is really worth to open a new issue on this) |
@tschoartschi latest version now published thanks for the heads up. Back home now so will likely work on this soon. |
my builds are breaking again, the .eslintignore is not being applied to the imported modules eg.
before the output was |
Try the beta again? |
I installed version So I get linting errors for example this file:
My eslintignore file looks as follows:
It was fine until I upgraded. What could be the issue? @jonathanKingston do you have any idea what I did wrong or what could cause the issue? |
@tschoartschi Check your glob syntax. I struggled with this for a bit and found out that was my problem. For example, instead of |
Should be fixed by ember-cli/broccoli-lint-eslint#30 |
Hi @BrianSipple @jonathanKingston can we get broccoli-lint-eslint bumped to get that change in? I can open the PR if needed. Thanks! |
I think the |
Is this still an issue? I am trying to exclude |
Yup, definitely still an issue. See #82 |
I have a script folder inside my ember cli project. I dont want to eslint this script folder because it is just quick&dirty try-out code of prototype-ideas. Now I put a .eslintignore file into my project root, right next to .eslintrc.
But the folder is still linted... what am I doing wrong?
Thanks
The text was updated successfully, but these errors were encountered: