Skip to content
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

allow using lower suffix for pascalCase in filename-case rule #395

Closed
alexander-akait opened this issue Sep 25, 2019 · 6 comments · Fixed by #431
Closed

allow using lower suffix for pascalCase in filename-case rule #395

alexander-akait opened this issue Sep 25, 2019 · 6 comments · Fixed by #431

Comments

@alexander-akait
Copy link

For example:

FooBar.test.js

Why?

  1. Some tools use/recommend lower notation for part after ., for example jest use lower test (FooBar.test.js) in examples for class testing.
  2. Sometimes you have vendor libraries (not in node_modules) and they use notation Timer.min.js (rare use case)
  3. Code for multiple platform, for example: MyClass.web.js, MyClass.android.js (react/react-native/other same tools)

Or maybe option to configure this

@fisker
Copy link
Collaborator

fisker commented Sep 26, 2019

how about a whitelist?

@alexander-akait
Copy link
Author

Sounds good, maybe will be great allow whitelist using regexp

@alexander-akait
Copy link
Author

WIP on the whitelist option.

Other case - i use eslint-plugin-markdown for linting code in Markdown files (using kebabCase and pascalCase values) so the filename-case rule report on some files, for example .github/ISSUE_TEMPLATE can be written in kebabCase or pascalCase.

Also many developers use README.md and CHANGELOG.md as standard practice.

@alexander-akait
Copy link
Author

Maybe we should use ignore name because eslint uses ignore as options in some rules, anyway it will be easy to rename

@fisker
Copy link
Collaborator

fisker commented Nov 8, 2019

ignore sounds better

@sindresorhus
Copy link
Owner

Let’s go with ignore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants