Use @tanstack/cli for creating Tanstack Start application#6829
Use @tanstack/cli for creating Tanstack Start application#6829rxssula wants to merge 1 commit intoTanStack:mainfrom
Conversation
📝 WalkthroughWalkthroughUpdated TanStack Start CLI command syntax in React documentation files. The recommended project creation command changed from Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/start/framework/react/quick-start.md`:
- Around line 10-18: The two fenced command blocks in the React quick-start
markdown are missing language identifiers; update both triple-backtick fences
around the commands "pnpx `@tanstack/cli` create my-app" and "npx `@tanstack/cli`
create my-app" to use ```bash so they include the bash language identifier
(e.g., change ``` to ```bash for each block) to satisfy markdownlint MD040 and
improve rendering consistency.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: f101422b-ff58-41f9-8063-ae50c73c9a88
📒 Files selected for processing (2)
docs/start/framework/react/getting-started.mddocs/start/framework/react/quick-start.md
| ``` | ||
| pnpm create @tanstack/start@latest | ||
| pnpx @tanstack/cli create my-app | ||
| ``` | ||
|
|
||
| or | ||
|
|
||
| ``` | ||
| npm create @tanstack/start@latest | ||
| npx @tanstack/cli create my-app | ||
| ``` |
There was a problem hiding this comment.
Add language identifiers to fenced command blocks.
Both command snippets are missing a fence language (bash), which triggers markdownlint MD040 and hurts rendering consistency.
Suggested patch
-```
+```bash
pnpx `@tanstack/cli` create my-app@@
- +bash
npx @tanstack/cli create my-app
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ``` | |
| pnpm create @tanstack/start@latest | |
| pnpx @tanstack/cli create my-app | |
| ``` | |
| or | |
| ``` | |
| npm create @tanstack/start@latest | |
| npx @tanstack/cli create my-app | |
| ``` |
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 10-10: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 16-16: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/start/framework/react/quick-start.md` around lines 10 - 18, The two
fenced command blocks in the React quick-start markdown are missing language
identifiers; update both triple-backtick fences around the commands "pnpx
`@tanstack/cli` create my-app" and "npx `@tanstack/cli` create my-app" to use
```bash so they include the bash language identifier (e.g., change ``` to
```bash for each block) to satisfy markdownlint MD040 and improve rendering
consistency.
Bundle Size Benchmarks
Trend sparkline is historical gzip bytes ending with this PR measurement; lower is better. |
Replace deprecated
npm create @tanstack/start@latestwithnpx @tanstack/cli create my-appSummary by CodeRabbit