Skip to content

Conversation

@michaelfaith
Copy link
Contributor

This change adds more details about prior versions of the plugin's config, to help people as they migrate from legacy to flat configs across multiple versions of this plugin. At some point in the 6.0 or 7.0 cycle, it would probably make sense to re-consolidate this into a single version.

Closes #32494

…sions

This change adds more details about prior versions of the config, to help people as they migrate from legacy to flat configs across multiple versions of this plugin. At some point in the 6.0 or 7.0 cycle, it would probably make sense to re-consolidate this into a single version.
@react-sizebot
Copy link

react-sizebot commented Mar 2, 2025

Comparing: 2980f27...df6bb59

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 518.24 kB 518.24 kB = 92.43 kB 92.43 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 570.57 kB 570.57 kB = 101.56 kB 101.56 kB
facebook-www/ReactDOM-prod.classic.js = 638.06 kB 638.06 kB = 112.28 kB 112.28 kB
facebook-www/ReactDOM-prod.modern.js = 628.38 kB 628.38 kB = 110.70 kB 110.70 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 693d4f6

@michaelfaith michaelfaith marked this pull request as ready for review March 2, 2025 21:17
Comment on lines 23 to 34
#### >= 6.0.0

For users of 6.0 and beyond, simply add the `recommended` config.

```js
import * as reactHooks from 'eslint-plugin-react-hooks';

export default [
// ...
reactHooks.configs.recommended,
];
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I think it might be confusing to have instructions for 6.0.0 when it's not released yet. I'd suggest removing this altogether and we can add it back just as we're about to publish.

Copy link
Contributor Author

@michaelfaith michaelfaith Mar 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed. Do you want me to go ahead and draft the follow-up change after this merges?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure!

@poteto poteto merged commit 443b7ff into facebook:main Mar 4, 2025
194 checks passed
@michaelfaith michaelfaith deleted the docs/hooks-additional-config-docs branch March 6, 2025 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

4 participants