warning File ignored by default. Use "--ignore-pattern '!node_modules/*'" to override

Yeah linting is still working but those warnings cause the test suite to fail.

On 18 May 2016, at 07:51, Brian Sipple notifications@github.com wrote:

It looks like this traces back to a behavior in eslint where warning messages for default-ignored files were being output. But then it also appears that this behavior has been removed as of eslint@2.10.2. I’m thinking we might just need to update the version of eslint being used under the hood by broccoli-lint-eslint – or configure it in a similar manner.

As for the file generation discrepancies, @oligriffiths, .eslintrc.js and tests/.eslintrc.js are now the intended starting points. I added a note to CHANGELOG.md to clarify that.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub

@maoueh Yep, I’m seeing this too.

This is a tough one. It appears to be related to an issue that’s affecting all users of eslint at the moment, and I’m not quite sure it’s something we can address here (But I’d love to hear if anyone has ideas 😀).

To elaborate, briefly: Behind the scenes, ember-cli-eslint runs eslint by way of broccoli-lint-eslint – which explicitly takes measures to prevent this from occurring. With atom’s linter-eslint, however, your linting pipeline is actually linter-eslint —> eslint, and thus it’s completely outside of anything ember-cli-eslint is doing.

(Worth noting, though — especially for anyone just now getting to this issue: since the error is localized to Atom, it shouldn’t cause builds to break, as that process does still go through broccoli-lint-eslint.)