Skip to content

Hook the default-app-setup OnLoad.cpp file with the cxxModuleProvider from RNCLI - #43049

Closed
cortinico wants to merge 2 commits into
react:mainfrom
cortinico:export-D53812109
Closed

Hook the default-app-setup OnLoad.cpp file with the cxxModuleProvider from RNCLI#43049
cortinico wants to merge 2 commits into
react:mainfrom
cortinico:export-D53812109

Conversation

@cortinico

Copy link
Copy Markdown
Contributor

Summary:
This connects the OnLoad.cpp file used by OSS apps with the rncli_cxxModuleProvider.
This method is created by the CLI and takes care of querying all the TM CXX Modules discovered and returning them.

Changelog:
[Internal] [Changed] - Hook the default-app-setup OnLoad.cpp file with the cxxModuleProvider from RNCLI

Differential Revision: D53812109

@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 Feb 15, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@analysis-bot

analysis-bot commented Feb 15, 2024

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a17,885,750+5
androidhermesarmeabi-v7an/a--
androidhermesx86n/a--
androidhermesx86_64n/a--
androidjscarm64-v8a21,240,097+11
androidjscarmeabi-v7an/a--
androidjscx86n/a--
androidjscx86_64n/a--

Base commit: 3a1e769
Branch: main

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@szymonrybczakszymonrybczak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

szymonrybczakand others added 2 commits February 19, 2024 07:44
Summary:
Bump CLI version to 13.6.0
## Changelog:
[GENERAL] [CHANGED] - Bump CLI to 13.6.0
Pull Request resolved: react#43093
Test Plan: CI
Differential Revision: D53920559
Pulled By: cortinico
… from RNCLI (react#43049)
Summary:
Pull Request resolved: react#43049
This connects the OnLoad.cpp file used by OSS apps with the `rncli_cxxModuleProvider`.
This method is created by the CLI and takes care of querying all the TM CXX Modules discovered and returning them.
This PR is currently waiting on react-native-community/cli#2296
Changelog:
[Internal] [Changed] - Hook the default-app-setup OnLoad.cpp file with the cxxModuleProvider from RNCLI
Reviewed By: cipolleschi
Differential Revision: D53812109
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label Feb 20, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in 4ecf57e.

huntie pushed a commit that referenced this pull request Feb 22, 2024
… from RNCLI (#43049)
Summary:
Pull Request resolved: #43049
This connects the OnLoad.cpp file used by OSS apps with the `rncli_cxxModuleProvider`.
This method is created by the CLI and takes care of querying all the TM CXX Modules discovered and returning them.
This PR is currently waiting on react-native-community/cli#2296
Changelog:
[Internal] [Changed] - Hook the default-app-setup OnLoad.cpp file with the cxxModuleProvider from RNCLI
Reviewed By: cipolleschi
Differential Revision: D53812109
fbshipit-source-id: 47bc0ea699516993070cfa0127de97853acf8890
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.fb-exportedMergedThis PR has been merged.p: FacebookPartner: FacebookPartner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@cortinico@facebook-github-bot@analysis-bot@szymonrybczak