Skip to content

[iOS][Local] Fix breaking changes in 0.74-rc.8 - #44025

Closed
cipolleschi wants to merge 1 commit into
0.74-stablefrom
cipolleschi/fix-brreaking-change
Closed

[iOS][Local] Fix breaking changes in 0.74-rc.8#44025
cipolleschi wants to merge 1 commit into
0.74-stablefrom
cipolleschi/fix-brreaking-change

Conversation

@cipolleschi

@cipolleschicipolleschi commented Apr 10, 2024

Copy link
Copy Markdown
Contributor

Summary:

With #43757, we pushed some breaking changes in RC.8

This change makes those changes not breaking.

Note (for framework developers): if you keep the old version of the APIs - which are now marked as deprecated - some feature (for example DeepLinking) won't be working, because they rely on the new parameters that needs to be passed to RCTHost.

Changelog:

[iOS][Fixed] - Make the changes from #43757 not breaking

Test Plan:

Tested locally

@facebook-github-botfacebook-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 labels Apr 10, 2024
@cipolleschi
cipolleschi changed the base branch from main to 0.74-stableApril 10, 2024 16:31
@cipolleschi
cipolleschiforce-pushed the cipolleschi/fix-brreaking-change branch from 7b47ef5 to eb7c552CompareApril 10, 2024 16:38
@cipolleschi
cipolleschi marked this pull request as ready for review April 10, 2024 16:38
@analysis-bot

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a17,820,459-1,580,414
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a21,178,721-1,597,461
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: 3f2b1d7
Branch: main

@cipolleschi

cipolleschi commented Apr 10, 2024

Copy link
Copy Markdown
ContributorAuthor

the failing gh action is okay. It is pulling hermes from main, but, of course, it can't work when the target branch is a stable one, which expect hermes from the stable version.

@cortinico

Copy link
Copy Markdown
Contributor

Closing as no longer necessary

@cortinico
cortinico deleted the cipolleschi/fix-brreaking-change branch June 10, 2024 13:12
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.p: FacebookPartner: FacebookPartnerPick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@cipolleschi@analysis-bot@cortinico@facebook-github-bot