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

Rename jSEngineInstance -> jsEngineInstance #38909

Closed
wants to merge 5 commits into from

Conversation

mdvacca
Copy link
Contributor

@mdvacca mdvacca commented Aug 9, 2023

Summary:
Rename jSEngineInstance -> jsEngineInstance

changelog: [internal] internal

Reviewed By: christophpurrer

Differential Revision: D48051751

@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 fb-exported labels Aug 9, 2023
@facebook-github-bot
Copy link
Contributor

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

@analysis-bot
Copy link

analysis-bot commented Aug 9, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,891,744 +202
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 9,483,438 +264
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 28dd1bb
Branch: main

mdvacca and others added 5 commits August 9, 2023 22:22
…eact.bridgeless

Summary:
in this diff I'm moving BridgelessReactPackage from com.facebook.react to com.facebook.react.bridgeless.

the goal is to hide this class from the API

changelog: [internal] internal

Differential Revision: D48050453

fbshipit-source-id: c4c0e178b597feaea11c6797edbce0ea574d4b85
Summary:
Make BridgelessReactPackage package only

changelog: [internal] internal

Differential Revision: D48050454

fbshipit-source-id: c061427440226c32c800a320f385b6e57e1ca7ba
Summary:
Rename BridgelessReactPackage to CoreReactPackage

changelog: [internal] internal

Differential Revision: D48050452

fbshipit-source-id: 9cfcc4cde135c28540a19c8360e911e67f55cba7
Summary:
Rename jSMainModulePath -> jsMainModulePath

changelog: [internal] internal

Differential Revision: D48051753

fbshipit-source-id: 51d74ad7c307dfe54e66855fba5da0d7bc31bba3
Summary:
Pull Request resolved: facebook#38909

Rename jSEngineInstance -> jsEngineInstance

changelog: [internal] internal

Reviewed By: christophpurrer

Differential Revision: D48051751

fbshipit-source-id: feee2898e260fe944fe0b5a8a99f9456af3851cf
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Aug 10, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e2ef6b9.

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