Skip to content

Commit 76cd503

Browse files
j-piaseckifacebook-github-bot
authored andcommitted
Drop unused Hermes import from RCTAppSetupUtils (#58056)
Summary: `RCTAppSetupUtils.h` imports `<reacthermes/HermesExecutorFactory.h>` but names nothing from it, so the import and the conditional that wrapped only it are removed. This unblocks classifying the Hermes executor as a private target under the three-tier C++ stable API visibility model, which it cannot be while a public header reaches it. Code that relied on reaching `HermesExecutorFactory` through this header should import it directly. Changelog: [iOS][Changed] - `RCTAppSetupUtils.h` no longer transitively imports `HermesExecutorFactory.h` Differential Revision: D116934198
1 parent ba1271d commit 76cd503

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

packages/react-native/Libraries/AppDelegate/RCTAppSetupUtils.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,6 @@
1313

1414
#import <memory>
1515

16-
#if USE_THIRD_PARTY_JSC != 1
17-
#import <reacthermes/HermesExecutorFactory.h>
18-
#endif
19-
2016
#import <ReactCommon/RCTTurboModuleManager.h>
2117
#import <jsireact/JSIExecutor.h>
2218

0 commit comments

Comments
 (0)