Skip to content

Conversation

cpojer
Copy link
Member

@cpojer cpojer commented Apr 22, 2025

6.0.0 of the ESLint plugin was released accidentally as per (facebook/react#32982 (comment)). However, when using a ^ for the version range, 6.0.0 takes precedence over 6.0.0-rc.1.

This PR fixes the blog post to lock the version to 6.0.0-rc.1.

If possible, please release 6.1.0-rc.1 of the plugin instead to ensure 6.0.0 will not accidentally be installed.

@cpojer
Copy link
Member Author

cpojer commented Apr 22, 2025

Also added a sentence explaining that the rule needs to be explicitly enabled since it is not enabled by default (see https://github.com/facebook/react/blob/main/packages/eslint-plugin-react-hooks/src/index.ts#L21-L24) and the rule's existence is not mentioned in the README of the package.

It would be great to add a strict config to the ESLint plugin that will enable and error on all rules.

@cpojer
Copy link
Member Author

cpojer commented Apr 22, 2025

cc @poteto @josephsavona

@poteto
Copy link
Member

poteto commented Apr 23, 2025

Thanks! Good call about rereleasing, I hadn't realized the caret range would pick up a deprecated package

@poteto poteto merged commit 39b9c56 into reactjs:main Apr 23, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants