-
Notifications
You must be signed in to change notification settings - Fork 24k
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
Update CLI to v12.1.1 #41530
Update CLI to v12.1.1 #41530
Conversation
packages/react-native/cli.js
Outdated
@@ -67,7 +67,7 @@ async function main() { | |||
// Ignore errors, since it's a nice to have warning | |||
} | |||
} | |||
return cli.run(); | |||
return cli.run('react-native'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More context: react-native-community/cli#2170
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's use name from package.json that way each out of tree platform will work out of the box
@@ -67,7 +67,7 @@ async function main() { | |||
// Ignore errors, since it's a nice to have warning | |||
} | |||
} | |||
return cli.run(); | |||
return cli.run(name); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, this change is intended to reduce fork surface for out of tree platforms like tvos, visionos, macos, etc, that would like to leverage CLI's init
command without modifications.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With this change, OOT platforms will only need to adjust template files and publish a package, and init
command will work.
56357d5
to
01f0023
Compare
01f0023
to
09c77b1
Compare
Base commit: 9b1f8a8 |
Summary: To unlock ~~certain OOT platform capabilities~~ seamless `init` integration for out-of-tree platforms with CLI, we need to pass the package name to it. This change landed on 0.73 branch already: #41530 Depends on #41722 ## Changelog: [INTERNAL] [ADDED] - Fix init for out-of-tree platforms by passing name to CLI Pull Request resolved: #41723 Test Plan: CI green Reviewed By: christophpurrer Differential Revision: D51979329 Pulled By: dmytrorykun fbshipit-source-id: 451f70dc42ae0667bc65cba2e77898c9eec8d9ec
…1723) Summary: To unlock ~~certain OOT platform capabilities~~ seamless `init` integration for out-of-tree platforms with CLI, we need to pass the package name to it. This change landed on 0.73 branch already: facebook#41530 Depends on facebook#41722 ## Changelog: [INTERNAL] [ADDED] - Fix init for out-of-tree platforms by passing name to CLI Pull Request resolved: facebook#41723 Test Plan: CI green Reviewed By: christophpurrer Differential Revision: D51979329 Pulled By: dmytrorykun fbshipit-source-id: 451f70dc42ae0667bc65cba2e77898c9eec8d9ec
Summary:
Updates CLI. Also I provided
platformName
parameter torun()
function when executing CLI.Changelog:
[GENERAL] [CHANGED] - Bump CLI to v12.1.1
Test Plan:
CI Green