-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Core #27
base: master
Are you sure you want to change the base?
Core #27
Conversation
…e, prettier) - Remove all configs except as needed to cover project-defined rules - Testing: Change from broken use of "category" to relying on custom `meta.version` for determination of ES version (eslint docs allowing for customization within `meta`) - Linting as per latest prettier - Update devDeps
Btw, while there were some other aspects of your config which might be of general interest, such as:
...I didn't add these as:
|
@mysticatea : Hi... Was just wondering if you were open in principle to this concept of a separate core plugin having the custom rules only (also asked for in #23) from which |
- Add coverage to ignore and lint all by default (adding to ignore allows discovery by IDEs)
I've published my own @brettz9/eslint-plugin which just extracts the bare rules (and the es5 and es6 configs), so if you aren't planning on developing this further on your own, you're of course welcome to close. |
Fixes #21 and #23
meta.version
for determination of ES version (eslint docs allowing for customization withinmeta
)Note that this is not a PR to be accepted here, but a PR to suggest a separate repository,
@mysticatea/eslint-plugin-core
(or however you'd want to name it). That is why I've submitted it as a draft, but it should be ready for review.If you are amenable to setting up such a separate repository, e.g., with the contents of this PR, I could look to amending the current code of
@mysticatea/eslint-plugin
proper to make use of this proposed@mysticatea/eslint-plugin-core
project.