-
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
delete RCTJSScriptLoaderModule #37284
Conversation
This pull request was exported from Phabricator. Differential Revision: D45591882 |
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 79bb8b11af7d912b54c0a19572100f76f0938a47
This pull request was exported from Phabricator. Differential Revision: D45591882 |
Base commit: d24f568 |
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 610c401d6c2f48c5a29258e8f7a4bf2e72d280dd
This pull request was exported from Phabricator. Differential Revision: D45591882 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D45591882 |
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 0d55af77edd18d6da2a9d6b21746d2d85cd4e6d0
This pull request was exported from Phabricator. Differential Revision: D45591882 |
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 1f0490a8c1f87065f31cf0e18363565a6eff3176
This pull request was exported from Phabricator. Differential Revision: D45591882 |
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 786486fd76d8e5683c645f677a437dcd9e112189
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 39f4d640cebd98abe8b5046c5b6a88813289677c
This pull request was exported from Phabricator. Differential Revision: D45591882 |
Differential Revision: D45582533 fbshipit-source-id: fc3c8781b80c0e5c16da456b61ad9b8f227fcdc5
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: a366e445d246110f0dea76095ec0cf2682cbad0e
This pull request was exported from Phabricator. Differential Revision: D45591882 |
This pull request has been merged in 438f6cf. |
Summary: Pull Request resolved: facebook#37284 Changelog: [iOS][Removed] in the past, we had this protocol so we could decorate the `RCTDevSplitBundleLoader`, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used by `RCTDevSplitBundleLoader`, so i don't think this will be a major breaking change Reviewed By: christophpurrer Differential Revision: D45591882 fbshipit-source-id: 23656d0c04fa4e1c4cfc65b1052d4c3614ba8708
Summary:
Changelog: [iOS][Removed]
in the past, we had this protocol so we could decorate the
RCTDevSplitBundleLoader
, which was removed by motiz88. now that doesn't exist anymore, we should delete this protocol. i did a quick grep on github and this protocol was only used byRCTDevSplitBundleLoader
, so i don't think this will be a major breaking changeReviewed By: christophpurrer
Differential Revision: D45591882