-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Enable more rules from the import plugin #947
Comments
Fortunately, it appears to have been sped up. If we benchmark it we can consider re-enabling it. import-js/eslint-plugin-import#803 (comment) |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Add this to the list for Standard 14 (#1321)? |
I just re-enabled |
I’d be interested in import/order. |
I find |
Issue for |
Here's another one. Easy profit. |
What about |
There seems to be a lot of rules from the import plugin that aren't enable, I think that we should consider enabling them.
Currently the ones enabled are:
which leaves a lot of rules not enabled:
Rules
Static analysis:
no-unresolved
)named
)default
)namespace
)no-restricted-paths
)no-absolute-path
)require()
calls with expressions (no-dynamic-require
)no-internal-modules
)no-webpack-loader-syntax
)Helpful warnings:
export
)no-named-as-default
)no-named-as-default-member
)@deprecated
documentation tag (no-deprecated
)no-extraneous-dependencies
)var
orlet
. (no-mutable-exports
)Module systems:
script
vs.module
) (unambiguous
)require
calls andmodule.exports
orexports.*
. (no-commonjs
)require
anddefine
calls. (no-amd
)no-nodejs-modules
)Style guide:
first
)no-duplicates
)no-namespace
)extensions
)order
)newline-after-import
)prefer-default-export
)max-dependencies
)no-unassigned-import
)no-named-default
)no-anonymous-default-export
)The text was updated successfully, but these errors were encountered: