Skip to content

Commit

Permalink
Don't ignore fixture directories by default (#317)
Browse files Browse the repository at this point in the history
Initially, I thought it would be a good idea, but turns out I actually want to lint most fixture files too.

For example, in AVA, we ended up having to explicitly specify it: https://github.com/avajs/ava/blob/a76d462b4cdccd86028e8c9b1bbf84f12af8ab84/package.json#L13

Another example, here it would also have been useful to lint fixtures: sindresorhus/electron-timber#10 (comment)
  • Loading branch information
sindresorhus committed Jun 24, 2018
1 parent 957b0d9 commit 00d6b7c
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions lib/options-manager.js
Expand Up @@ -18,9 +18,6 @@ const DEFAULT_IGNORE = [
'{tmp,temp}/**',
'**/*.min.js',
'**/bundle.js',
'fixture{-*,}.{js,jsx}',
'fixture{s,}/**',
'{test,tests,spec,__tests__}/fixture{s,}/**',
'vendor/**',
'dist/**'
];
Expand Down

0 comments on commit 00d6b7c

Please sign in to comment.