-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
Hook the default-app-setup OnLoad.cpp file with the cxxModuleProvider from RNCLI #43049
Conversation
This pull request was exported from Phabricator. Differential Revision: D53812109 |
Base commit: 3a1e769 |
...e-gradle-plugin/src/test/kotlin/com/facebook/react/tasks/GenerateCodegenArtifactsTaskTest.kt
Outdated
Show resolved
Hide resolved
8bc8cc3
to
7f8d9f0
Compare
… from RNCLI (facebook#43049) 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. 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
This pull request was exported from Phabricator. Differential Revision: D53812109 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
This pull request was exported from Phabricator. Differential Revision: D53812109 |
… from RNCLI (facebook#43049) Summary: Pull Request resolved: facebook#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
7f8d9f0
to
1340378
Compare
Summary: Bump CLI version to 13.6.0 ## Changelog: [GENERAL] [CHANGED] - Bump CLI to 13.6.0 Pull Request resolved: facebook#43093 Test Plan: CI Differential Revision: D53920559 Pulled By: cortinico
… from RNCLI (facebook#43049) Summary: Pull Request resolved: facebook#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
This pull request was exported from Phabricator. Differential Revision: D53812109 |
1340378
to
bc9f00e
Compare
This pull request has been merged in 4ecf57e. |
… 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
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