Add ignore option to the filename-case rule
#431
Conversation
|
I think you we need regexp support, for examples |
alexander-akait
force-pushed
the
feat-ignore-option-for-the-filename-case-rule
branch
from
a172098 to
2c2889a
Compare
|
@fisker done |
|
strange problem with integration tests, what tips will be? |
|
something in |
sindresorhus
changed the title
ignore option for the filename-case ruleignore option for the filename-case rule
|
@sindresorhus I'm updating dependencies |
| testCase('src/foo/{foo_bar}.js', 'snakeCase') | ||
| testCase('src/foo/{foo_bar}.js', 'snakeCase'), | ||
| testCaseWithOptions(undefined, undefined, [ | ||
| {case: 'kebabCase', ignore: ['FOOBAR\\.js']} |
There was a problem hiding this comment.
It should allow specifying regex literals. Most people use JS configs, so it's nicer to also be able to specify literals.
There was a problem hiding this comment.
@sindresorhus just allow use /foobar\.js/ as regexp, right? When i use string notation foobar\\.js it is also regexp?
There was a problem hiding this comment.
Allow both both regex literal and regex in a string.
There was a problem hiding this comment.
is regex in a string a good idea? maybe confusing people
fisker
force-pushed
the
feat-ignore-option-for-the-filename-case-rule
branch
from
4e61bba to
55c617c
Compare
|
@sindresorhus @fisker done |
| @@ -68,11 +68,19 @@ You can set the `cases` option to allow multiple cases: | |||
|
|
|||
| ### ignore | |||
|
|
|||
| Type: `RegExp[]`\ | |||
| Type: `Array<string | RegExp>`\ | |||
| Default: `[]` | |||
|
|
|||
| Filenames to ignore. | |||
|
|
|||
There was a problem hiding this comment.
Can you explicitly say that if a string is given, it's interpreted as a regex inside a string.
|
so, to ignore |
|
Correct. It might be worth adding a tip about that to the docs, as a lot of user miss how to correctly escape regexes. |
|
@sindresorhus done, English is not my native so I can make a mistake somewhere 😄 |
sindresorhus
changed the title
ignore option for the filename-case ruleignore option to the filename-case rule
fixes #395