-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Hotfix to include react-native-windows in hasteImpl accepted paths #20007
Conversation
…acebook#20007) Summary: Closes facebook#20007 We removed support for providesModule annotations and maintained support for Haste names in installed modules via `providesModuleNodeModules`, but our default `hasteImpl` doesn't take them into account. We need to find a better way to override core components from plugins but meanwhile this adds an exception for react-native-windows in the default `hasteImpl` to unblock their upgrade to the latest RC. Fixes facebook/metro#188 Reviewed By: mjesun Differential Revision: D8695207 fbshipit-source-id: e70c8649543ebfcba40f934451faf33548c3d20a
Test passed. Closing. |
…20007) Summary: Closes #20007 We removed support for providesModule annotations and maintained support for Haste names in installed modules via `providesModuleNodeModules`, but our default `hasteImpl` doesn't take them into account. We need to find a better way to override core components from plugins but meanwhile this adds an exception for react-native-windows in the default `hasteImpl` to unblock their upgrade to the latest RC. Fixes facebook/metro#188 Reviewed By: mjesun Differential Revision: D8695207 fbshipit-source-id: 2ad6cb1e93e600880a148776ac45f6ebd7d205d3
This pull request was closed by @rubennorte in 5494274. Once this commit is added to a release, you will see the corresponding version tag below the description at 5494274. If the commit has a single |
…20007) Summary: Closes #20007 We removed support for providesModule annotations and maintained support for Haste names in installed modules via `providesModuleNodeModules`, but our default `hasteImpl` doesn't take them into account. We need to find a better way to override core components from plugins but meanwhile this adds an exception for react-native-windows in the default `hasteImpl` to unblock their upgrade to the latest RC. Fixes facebook/metro#188 Reviewed By: mjesun Differential Revision: D8695207 fbshipit-source-id: 2ad6cb1e93e600880a148776ac45f6ebd7d205d3
Thanks @hramos - what's the status update on a generalization of this? I still need a solution for the |
cc @rubennorte, original author of the diff that synced out as 5494274. |
…20007) Summary: Closes facebook/react-native#20007 We removed support for providesModule annotations and maintained support for Haste names in installed modules via `providesModuleNodeModules`, but our default `hasteImpl` doesn't take them into account. We need to find a better way to override core components from plugins but meanwhile this adds an exception for react-native-windows in the default `hasteImpl` to unblock their upgrade to the latest RC. Fixes facebook/metro#188 Reviewed By: mjesun Differential Revision: D8695207 fbshipit-source-id: 2ad6cb1e93e600880a148776ac45f6ebd7d205d3
Summary:
We removed support for providesModule annotations and maintained support for Haste names in installed modules via
providesModuleNodeModules
, but our defaulthasteImpl
doesn't take them into account. We need to find a better way to override core components from plugins but meanwhile this adds an exception for react-native-windows in the defaulthasteImpl
to unblock their upgrade to the latest RC.Fixes facebook/metro#188
Reviewed By: mjesun
Differential Revision: D8695207