-
Notifications
You must be signed in to change notification settings - Fork 24.1k
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
Upstream autolinking improvements to React Native Core #45302
Comments
|
Summary: Fixes facebook#45302 I'm ustreaming two changes that happened inside the CLI/Autolinking logic that are missing inside core autolinking. Changelog: [Internal] [Changed] - Upstream CLI autolinking changes to RNGP Differential Revision: D59460504
Hey @szymonrybczak, so I could easily upstream react-native-community/cli#2387 The other PR however is a bit more complicated (react-native-community/cli#2379). The reason is that by default, the Android template will invoke As |
Summary: Pull Request resolved: facebook#45314 Fixes facebook#45302 I'm ustreaming two changes that happened inside the CLI/Autolinking logic that are missing inside core autolinking. Changelog: [Internal] [Changed] - Upstream CLI autolinking changes to RNGP Reviewed By: blakef Differential Revision: D59460504
Summary: Pull Request resolved: facebook#45314 Fixes facebook#45302 I'm ustreaming two changes that happened inside the CLI/Autolinking logic that are missing inside core autolinking. Changelog: [Internal] [Changed] - Upstream CLI autolinking changes to RNGP Reviewed By: blakef Differential Revision: D59460504
But we can land this change in |
@szymonrybczak I'm not sure I follow you |
@cortinico Hm, maybe I didn't understand.
If latest |
That's a good question :) not sure why it's failing |
@cortinico could you send me a job that failed, I can take a look :) |
Summary: This is a followup for facebook#45302 Changelog: [Internal] [Changed] - Add --platform android to default config invocation for autolinking Differential Revision: D59577996
@szymonrybczak here it is: #45360 |
@cortinico thank you for working on these! 🙏 |
Summary: This is a followup for facebook#45302 Changelog: [Internal] [Changed] - Add --platform android to default config invocation for autolinking Differential Revision: D59577996
Summary: Pull Request resolved: #45314 Fixes #45302 I'm ustreaming two changes that happened inside the CLI/Autolinking logic that are missing inside core autolinking. Changelog: [Internal] [Changed] - Upstream CLI autolinking changes to RNGP Reviewed By: blakef Differential Revision: D59460504 fbshipit-source-id: a6e7ab6a46fec26620dc965e77b13fdeba84fae3
Description
Recently we've landed few improvements to autolinking mechanism, but since autolinking was moved to Core we need to replicate these fixes:
--platform
with which only platform's specific Config is fetched.cc: @cortinico
The text was updated successfully, but these errors were encountered: