Skip to content
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 create-expo-app CLI help screen and README to reflect new default template in v51 #28738

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

zeckdude
Copy link

@zeckdude zeckdude commented May 10, 2024

Why

As mentioned at https://expo.dev/changelog/2024/05-07-sdk-51#new-default-project-template-and-%22getting-started%22-flow, the default template changed from 'blank' to 'tabs'. This change just fixes the CLI help screen and README to reflect that.

How

Just fixed the instructions text

Test Plan

Checklist

As mentioned at https://expo.dev/changelog/2024/05-07-sdk-51#new-default-project-template-and-%22getting-started%22-flow, the default template changed from 'blank' to 'tabs'. This change just fixes the CLI help screen to reflect that.
@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label May 10, 2024
@zeckdude zeckdude changed the title Update cli.ts to reflect new default template in v51 Update create-expo-app CLI and README instructions to reflect new default template in v51 May 10, 2024
@zeckdude zeckdude changed the title Update create-expo-app CLI and README instructions to reflect new default template in v51 Update create-expo-app CLI help screen and README instructions to reflect new default template in v51 May 10, 2024
@zeckdude zeckdude changed the title Update create-expo-app CLI help screen and README instructions to reflect new default template in v51 Update create-expo-app CLI help screen and README to reflect new default template in v51 May 10, 2024
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels May 10, 2024
Copy link
Member

@brentvatne brentvatne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for pointing this out! i completely missed it. made some changes to your suggestion because we actually use the default template and not tabs (we may drop the tabs template at some point)

packages/create-expo/README.md Outdated Show resolved Hide resolved
packages/create-expo/src/cli.ts Outdated Show resolved Hide resolved
@@ -38,7 +38,7 @@ async function run() {
[
`-y, --yes Use the default options for creating a project`,
` --no-install Skip installing npm packages or CocoaPods`,
chalk`-t, --template {gray [pkg]} NPM template to use: blank, tabs, bare-minimum. Default: blank`,
chalk`-t, --template {gray [pkg]} NPM template to use: default, blank, tabs, bare-minimum.`,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brentvatne maybe we should omit this, we also support github URLs or tarbal paths with --template. That's not reflected here at all either.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't the more appropriate solution be to include all options here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants