Skip to content

[flow] Replace $PropertyType with indexed access type in ReactNativeTypes #32733

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

Merged
merged 1 commit into from
Mar 25, 2025

Conversation

SamChou19815
Copy link
Contributor

Summary

Replace these deprecated types to prepare for removal of support in Flow.

How did you test this change?

flow

…ypes

Replace these deprecated types to prepare for removal of support in Flow.
@react-sizebot
Copy link

Comparing: e5f275e...5c5dda8

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 +0.05% 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 515.14 kB 515.14 kB = 91.74 kB 91.74 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 = 614.35 kB 614.35 kB = 108.76 kB 108.76 kB
facebook-www/ReactDOM-prod.classic.js = 650.34 kB 650.34 kB = 114.79 kB 114.79 kB
facebook-www/ReactDOM-prod.modern.js = 640.62 kB 640.62 kB = 113.22 kB 113.22 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
react-native/shims/ReactNativeTypes.js = 7.27 kB 7.24 kB = 2.00 kB 2.00 kB

Generated by 🚫 dangerJS against 5c5dda8

@SamChou19815 SamChou19815 marked this pull request as ready for review March 24, 2025 23:50
Copy link
Contributor

@jbrown215 jbrown215 left a comment

Choose a reason for hiding this comment

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

send it!

@SamChou19815 SamChou19815 merged commit b59f186 into main Mar 25, 2025
245 checks passed
@SamChou19815 SamChou19815 deleted the replace-dollar-property-type branch March 25, 2025 03:00
github-actions bot pushed a commit that referenced this pull request Mar 25, 2025
github-actions bot pushed a commit that referenced this pull request Mar 25, 2025
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Mar 26, 2025
Summary:
Pull Request resolved: #50271

In the next version of Flow, support for `$PropertyType` will be removed in favor of indexed access types. The final 2 usages in react-native is in `ReactNativeTypes`, which is synced from upstream react. Sync of facebook/react#32733 is currently blocked by test failures, so I will add the temporary shim to unblock releases.

Changelog: [Internal]

Reviewed By: panagosg7

Differential Revision: D71849353

fbshipit-source-id: d4787fe17ae52aa50d54f9fb7fb8b14c149aa1e6
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