Skip to content

Transform $ReadOnly to Readonly 38/n - #55067

Closed
marcoww6 wants to merge 2 commits into
react:mainfrom
marcoww6:export-D90146229
Closed

Transform $ReadOnly to Readonly 38/n#55067
marcoww6 wants to merge 2 commits into
react:mainfrom
marcoww6:export-D90146229

Conversation

@marcoww6

@marcoww6 marcoww6 commented Jan 6, 2026

Copy link
Copy Markdown
Contributor

Differential Revision: D90146229

Marco Wang and others added 2 commits January 6, 2026 14:19
Summary:
We are transforming the following utility types to be more consistent with typescript and better AI integration:

* `$NonMaybeType` -> `NonNullable`
* `$ReadOnly` -> `Readonly`
* `$ReadOnlyArray` -> `ReadonlyArray`
* `$ReadOnlyMap` -> `ReadonlyMap`
* `$ReadOnlySet` -> `ReadonlySet`
* `$Keys` -> `keyof`
* `$Values` -> `Values`
* `mixed` -> `unknown`

See details in https://fb.workplace.com/groups/flowlang/permalink/1837907750148213/.

drop-conflicts

Command:

`js1 flow-runner codemod flow/transformUtilityType --legacy-type='$ReadOnly'`

Differential Revision: D90146145
Differential Revision: D90146229
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 6, 2026
@meta-codesync

meta-codesync Bot commented Jan 6, 2026

Copy link
Copy Markdown

@marcoww6 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D90146229.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @marcoww6 in 41efc3b

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants