-
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
Reduce visibility of JSCJavaScriptExecutorFactory #39737
Conversation
This pull request was exported from Phabricator. Differential Revision: D49752141 |
This pull request was exported from Phabricator. Differential Revision: D49752141 |
Summary: Pull Request resolved: facebook#39737 In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory Reviewed By: RSNara Differential Revision: D49752141 fbshipit-source-id: 2dbf2537d9f53a5aa89ab829a6d5fbe7a4cb6351
c2d9950
to
67802fa
Compare
Base commit: 294c0fa |
This pull request was exported from Phabricator. Differential Revision: D49752141 |
Summary: Pull Request resolved: facebook#39737 In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory Reviewed By: RSNara Differential Revision: D49752141 fbshipit-source-id: 0431b871f2243e161c00390e26ad7d289d7b680c
67802fa
to
12861ae
Compare
This pull request was exported from Phabricator. Differential Revision: D49752141 |
Summary: Pull Request resolved: facebook#39737 In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory Reviewed By: RSNara Differential Revision: D49752141 fbshipit-source-id: 3f8a7a34cc1779aacb86a551e675876130173e70
12861ae
to
8290d1b
Compare
This pull request was exported from Phabricator. Differential Revision: D49752141 |
Summary: Pull Request resolved: facebook#39737 In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory Reviewed By: RSNara Differential Revision: D49752141 fbshipit-source-id: ed240fe590953414a64b4b4c4f3d5aedf4ecd0d9
8290d1b
to
ff6a009
Compare
Summary: In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of DynamicFromArray Differential Revision: D49752134 fbshipit-source-id: 42f72505d6163d0000b34151c1490ee7b980d343
Summary: In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of DynamicFromMap Differential Revision: D49752137 fbshipit-source-id: e3b7e346cc1745f9f1eca4b243c1b8c3ee48df88
This pull request was exported from Phabricator. Differential Revision: D49752141 |
Summary: Pull Request resolved: facebook#39737 In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory Reviewed By: RSNara Differential Revision: D49752141 fbshipit-source-id: 2982990eb2321b119213f6f7152785f24f222ab7
ff6a009
to
505c3c2
Compare
This pull request has been merged in 11d51fe. |
Summary:
In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages.
As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages.
If you are using this class or interface please comment in this PR and we will restate the public access.
changelog: [Android][Changed] Reducing visibility of JSCJavaScriptExecutorFactory
Reviewed By: RSNara
Differential Revision: D49752141