-
Notifications
You must be signed in to change notification settings - Fork 11
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
Prevent further prompts in npx create-react-native-library #156
Conversation
@@ -165,7 +165,7 @@ create_library() { | |||
example_type=vanilla | |||
fi | |||
echo "-- Creating library $PROJECT_SLUG with create-react-native-library@$BOB_VERSION" | |||
npx "create-react-native-library@$BOB_VERSION" \ | |||
npm_config_yes=true npx "create-react-native-library@$BOB_VERSION" \ |
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.
This confirms the install prompt when you haven't run the supplied version of create-react-native-library before.
Need to install the following packages:
[email protected]
Ok to proceed? (y)
@@ -175,6 +175,7 @@ create_library() { | |||
--languages cpp \ | |||
--type module-new \ | |||
--example $example_type \ | |||
--local false \ |
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.
This declines generating a local library when running the script from the root folder of another project (such as ubrn itself).
? Looks like you're under a project folder. Do you want to create a local library? › (Y/n)
There seems to be a timing issue on the CI that makes the abort-controller test flake occasionally. That should be unrelated to the changes in this PR though. |
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.
These changes seem sensible and reasonable. I have tested them on my local machine, and they work.
Great work, thank you once again @Johennes !
✅
These changes should make it easier to run
test-turbo-modules.sh
on the CI in future.