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

Fix RCTFabric podspec #42639

Closed
wants to merge 1 commit into from
Closed

Conversation

cipolleschi
Copy link
Contributor

Summary:
When I reverted part of the deprecation of RCT_NEW_ARCH_ENABLED, I forget a little bit which was breaking RCTFabric podspec.

This diff fixes that.

Changelog:

[Internal] - Bring back RCT_NEW_ARCH_ENABLED to Fabric to make the RCTThirdPartyFabricComponentsProvider work again.

Differential Revision: D53048270

@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 labels Jan 24, 2024
@facebook-github-bot
Copy link
Contributor

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

Summary:

When I reverted part of the deprecation of `RCT_NEW_ARCH_ENABLED`, I forget a little bit which was breaking RCTFabric podspec.

This diff fixes that.

## Changelog:
[Internal] - Bring back `RCT_NEW_ARCH_ENABLED` to Fabric to make the `RCTThirdPartyFabricComponentsProvider` work again.

Reviewed By: cortinico

Differential Revision: D53048270
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jan 25, 2024
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in f4dbd15.

@normtronics
Copy link

Did this get released?

@cipolleschi
Copy link
Contributor Author

Not yet, it will be included in 0.74.
If you need this picked in a previous version of React Native, can you open a pick request in the proper discussion?

lunaleaps pushed a commit that referenced this pull request Feb 20, 2024
Summary:
Pull Request resolved: #42639

When I reverted part of the deprecation of `RCT_NEW_ARCH_ENABLED`, I forget a little bit which was breaking RCTFabric podspec.

This diff fixes that.

[Internal] - Bring back `RCT_NEW_ARCH_ENABLED` to Fabric to make the `RCTThirdPartyFabricComponentsProvider` work again.

Reviewed By: cortinico

Differential Revision: D53048270

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