Skip to content

Conversation

poteto
Copy link
Member

@poteto poteto commented Oct 1, 2025

@meta-cla meta-cla bot added the CLA Signed label Oct 1, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Oct 1, 2025
@react-sizebot
Copy link

react-sizebot commented Oct 1, 2025

Comparing: 289f070...13d42de

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 = 536.14 kB 536.14 kB = 94.81 kB 94.81 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 = 663.96 kB 663.96 kB = 117.04 kB 117.04 kB
facebook-www/ReactDOM-prod.classic.js = 687.83 kB 687.83 kB = 121.08 kB 121.08 kB
facebook-www/ReactDOM-prod.modern.js = 678.26 kB 678.26 kB = 119.44 kB 119.43 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 13d42de

@cpojer
Copy link
Contributor

cpojer commented Oct 2, 2025

Oh, it's just gonna be a default, that's great!

Adds back the compiler rules to the recommended preset, intended for the next release.
@poteto poteto merged commit 5cc3d49 into main Oct 2, 2025
468 of 469 checks passed
@poteto poteto deleted the pr34675 branch October 2, 2025 22:42
poteto added a commit that referenced this pull request Oct 2, 2025
Updates the eslint fixture lockfiles.

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/34699).
* #34703
* #34700
* __->__ #34699
* #34675
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
Updates the eslint fixture lockfiles.

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/34699).
* #34703
* #34700
* __->__ #34699
* #34675

DiffTrain build for [056a586](056a586)
github-actions bot pushed a commit that referenced this pull request Oct 2, 2025
Updates the eslint fixture lockfiles.

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/34699).
* #34703
* #34700
* __->__ #34699
* #34675

DiffTrain build for [056a586](056a586)
@OliverJAsh
Copy link

I think the docs need updating. These rules are still listed as "additional" (rather than recommended) here: https://react.dev/reference/eslint-plugin-react-hooks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants