Uh oh!
There was an error while loading. Please reload this page.
feat: deprecate passing separate new arch flags to RCTRootViewFactory - #46652
feat: deprecate passing separate new arch flags to RCTRootViewFactory#46652okwasniewski wants to merge 1 commit into
Conversation
cipolleschi
left a comment
There was a problem hiding this comment.
Sadly, we can't merge this. If we do this, people will not be able to turn off selectively parts of the New Architecture, and we are not ready to remove this possibility just yet.
okwasniewski
commented
Sep 27, 2024
@cipolleschi Yeah, I was afraid its too early 😅 Do you think we can deprecate RCTRootViewFactory initializers? |
cipolleschi
commented
Oct 2, 2024
What do you mean? Do you want to move away from some initializers? |
okwasniewski
commented
Oct 2, 2024
@cipolleschi Yeah, I want to deprecate initializers that accept multiple flags Deprecate this: - (instancetype)initWithBundleURL:(NSURL *)bundleURL
newArchEnabled:(BOOL)newArchEnabled
turboModuleEnabled:(BOOL)turboModuleEnabled
bridgelessEnabled:(BOOL)bridgelessEnabledAnd make this preferred: - (instancetype)initWithBundleURL:(NSURL *)bundleURL
newArchEnabled:(BOOL)newArchEnabledSo users that rely on setting different flags can still use it but for new (brownfield) integrations they can use newArchEnabled parameter. |
cipolleschi
commented
Oct 2, 2024
Yeah, that seems reasonable to me. |
a873d9f to
688e923Compare688e923 to
4b4ae52Compareokwasniewski
commented
Oct 4, 2024
@cipolleschi I've deprecated the initializers |
facebook-github-bot
commented
Oct 10, 2024
@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
facebook-github-bot
commented
Oct 10, 2024
@cipolleschi merged this pull request in 7e1674f. |
react-native-bot
commented
Oct 10, 2024
This pull request was successfully merged by @okwasniewski in 7e1674f When will my fix make it into a release? | How to file a pick request? |
Summary:
This PR follows up with the deprecation introduced here: #46228
The idea is to have new architecture depend on one flag, namely
newArchEnabled. It exposes additional initializers for RCTRootViewFactory.Changelog:
[IOS] [CHANGED] - Use
newArchEnabledflag in RCTAppDelegate and RCTRootViewFactoryTest Plan:
CI Green