Skip to content
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

Codemod $Shape to Partial in xplat, suppressing errors [4] #36960

Closed
wants to merge 1 commit into from

Conversation

gkz
Copy link
Member

@gkz gkz commented Apr 18, 2023

Summary:
We're deprecating the unsafe $Shape and moving to the safe Partial: https://fb.workplace.com/groups/flowlang/posts/1251655088773485

I have previously codemodded all locations that do not cause errors. Now start on the remaining ones: codemod and suppress.

Changelog: [Internal]

Reviewed By: SamChou19815

Differential Revision: D45076273

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Apr 18, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45076273

@analysis-bot
Copy link

analysis-bot commented Apr 18, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,620,271 +0
android hermes armeabi-v7a 7,932,954 +0
android hermes x86 9,106,265 +0
android hermes x86_64 8,961,084 +0
android jsc arm64-v8a 9,186,874 +0
android jsc armeabi-v7a 8,377,164 +0
android jsc x86 9,244,456 +0
android jsc x86_64 9,502,971 +0

Base commit: 6f94cd2
Branch: main

…ook#36960)

Summary:
Pull Request resolved: facebook#36960

We're deprecating the unsafe `$Shape` and moving to the safe `Partial`: https://fb.workplace.com/groups/flowlang/posts/1251655088773485

I have previously codemodded all locations that do not cause errors. Now start on the remaining ones: codemod and suppress.

Changelog: [Internal]

Reviewed By: SamChou19815

Differential Revision: D45076273

fbshipit-source-id: 951db0f6d850c2c20a87c146370b27a565ae2248
@gkz gkz force-pushed the export-D45076273 branch from c0081bf to 43cb6af Compare April 18, 2023 19:18
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D45076273

facebook-github-bot pushed a commit to facebook/metro that referenced this pull request Apr 18, 2023
Summary:
X-link: facebook/react-native#36960

We're deprecating the unsafe `$Shape` and moving to the safe `Partial`: https://fb.workplace.com/groups/flowlang/posts/1251655088773485

I have previously codemodded all locations that do not cause errors. Now start on the remaining ones: codemod and suppress.

Changelog: [Internal]

Reviewed By: SamChou19815

Differential Revision: D45076273

fbshipit-source-id: 27ebf33370143e19751dbdcfcc1876cf3c586e14
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Apr 18, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e2116d2.

jeongshin pushed a commit to jeongshin/react-native that referenced this pull request May 7, 2023
…ook#36960)

Summary:
Pull Request resolved: facebook#36960

We're deprecating the unsafe `$Shape` and moving to the safe `Partial`: https://fb.workplace.com/groups/flowlang/posts/1251655088773485

I have previously codemodded all locations that do not cause errors. Now start on the remaining ones: codemod and suppress.

Changelog: [Internal]

Reviewed By: SamChou19815

Differential Revision: D45076273

fbshipit-source-id: 27ebf33370143e19751dbdcfcc1876cf3c586e14
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
…ook#36960)

Summary:
Pull Request resolved: facebook#36960

We're deprecating the unsafe `$Shape` and moving to the safe `Partial`: https://fb.workplace.com/groups/flowlang/posts/1251655088773485

I have previously codemodded all locations that do not cause errors. Now start on the remaining ones: codemod and suppress.

Changelog: [Internal]

Reviewed By: SamChou19815

Differential Revision: D45076273

fbshipit-source-id: 27ebf33370143e19751dbdcfcc1876cf3c586e14
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. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants