Skip to content

Turn on string ref deprecation warning for everybody (not codemoddable) - #25383

Merged
sebmarkbage merged 3 commits into
react:mainfrom
eps1lon:forwardRef/deprecate-string-refs-only
Nov 17, 2022
Merged

Turn on string ref deprecation warning for everybody (not codemoddable)#25383
sebmarkbage merged 3 commits into
react:mainfrom
eps1lon:forwardRef/deprecate-string-refs-only

Conversation

@eps1lon

@eps1lon eps1lon commented Oct 2, 2022

Copy link
Copy Markdown
Collaborator

Summary

Alternate to #25334 without any prod runtime changes i.e. the proposed codemod in https://github.com/reactjs/rfcs/blob/createlement-rfc/text/0000-create-element-changes.md#deprecate-string-refs-and-remove-production-mode-_owner-field would not work.

How did you test this change?

  • CI
  • yarn test with and without warnAboutStringRefs

@ghost ghost added the CLA Signed label Oct 2, 2022
@ghost
ghost force-pushed the forwardRef/deprecate-string-refs-only branch from ea0c6a3 to 8b20492 Compare October 2, 2022 14:38

ghost commented Oct 2, 2022

Copy link
Copy Markdown

Comparing: d65b88d...6f24ce6

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.min.js = 153.62 kB 153.62 kB = 48.83 kB 48.83 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 155.55 kB 155.55 kB = 49.47 kB 49.47 kB
facebook-www/ReactDOM-prod.classic.js = 530.40 kB 530.40 kB = 94.14 kB 94.14 kB
facebook-www/ReactDOM-prod.modern.js = 515.66 kB 515.66 kB = 91.95 kB 91.95 kB
facebook-www/ReactDOMForked-prod.classic.js = 530.40 kB 530.40 kB = 94.15 kB 94.14 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 6f24ce6

@ghost ghost changed the title Turn on string ref deprecation warning for everybody Turn on string ref deprecation warning for everybody (not codemoddable) Oct 9, 2022
@ghost
ghost force-pushed the forwardRef/deprecate-string-refs-only branch from 8b20492 to 062de60 Compare October 9, 2022 09:11
@ghost ghost mentioned this pull request Oct 9, 2022
1 task
This was referenced Nov 9, 2024
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.

4 participants