-
Notifications
You must be signed in to change notification settings - Fork 46.4k
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
Proposal of recommended config in eslint-plugin-react-hooks
#16872
Comments
I would also very much like this. Is it a matter of just putting the above in a PR, or would comprehensive tests also be needed? I'd expect the documentation should also be updated to recommend using this instead of manual rules. |
Does anyone know why the recommendation is to set exhaustive-deps to warn instead of error? |
I'm also struggling to understand this decision. Not adding a dependency seems like an extremely rare situation that warrants an |
"extends": [
"eslint:recommended",
"plugin:import/recommended",
"plugin:react/recommended",
"plugin:react-hooks/recommended" // proposal
] |
Actually I found out why |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contribution. |
Closing this issue after a prolonged period of inactivity. If this issue is still present in the latest release, please create a new issue with up-to-date information. Thank you! |
Now the extending `plugin:react-hooks/recommended` adds this plugin and configures both of its rules.
Now the extending `plugin:react-hooks/recommended` adds this plugin and configures both of its rules.
Now the extending `plugin:react-hooks/recommended` adds this plugin and configures both of its rules.
Do you want to request a feature or report a bug?
Feature
What is the current behavior?
If you are using
eslint-plugin-react-hooks
you need to put rules configuration manually. What if in the next version you will add some more rules?Should everyone who use it after future release add all the new rules to config manually?
What is the expected behavior?
I propose instead of instructing on every new rule to create
recommended
config as ineslint-plugin-react
, as follows:Also, to instruct to extend instead:
The text was updated successfully, but these errors were encountered: