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

docs: add commands in code groups to enhance clarity and ease of use #17396

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

Conversation

BhanuVikram
Copy link

Description

The guide is missing copiable commands for: 1. Scaffolding Vite inside an existing directory, and 2. Scaffolding Vite inside an existing directory along with a template. I have added the commands neatly in code groups to reduce confusion, improve presentation, and enhance clarity.

  1. Added commands (used to scaffold inside an existing directory) in a code group.

  2. Updated commands (used to specify a project name and template) in a code group.

  3. Added commands (used to scaffold with a template inside an existing directory) in a code group.

Added three code groups to enhance clarity and easy of use: 

1. Added commands (used to scaffold inside an existing directory) in a code group. 

2. Updated commands (used to specify a project name and template) in a code group. 

3. Added commands (used to scaffold with a template inside an existing directory) in a code group.
Copy link

stackblitz bot commented Jun 5, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Comment on lines +98 to 116
::: code-group

```bash [NPM]
# npm 7+, extra double-dash is needed:
npm create vite@latest my-vue-app -- --template vue
```

# yarn
```bash [Yarn]
yarn create vite my-vue-app --template vue
```

# pnpm
```bash [PNPM]
pnpm create vite my-vue-app --template vue
```

# bun
```bash [Bun]
bun create vite my-vue-app --template vue
```

Copy link
Member

Choose a reason for hiding this comment

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

Can we keep this change only and revert the rest? I don't think the others need to be added/changed.

Copy link
Author

Choose a reason for hiding this comment

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

Okay. That sounds good. If I see a need for any further changes, I will make a new PR.

@vitejs vitejs deleted a comment from pkg-pr-new bot Jun 13, 2024
Copy link

pkg-pr-new bot commented Jun 13, 2024

vite (3e27071)

npm i https://pkg.pr.new/vite@17396

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants