Skip to content

feat(create-vite): scaffold with rolldown-vite#20739

Merged
sapphi-red merged 8 commits into
vitejs:mainfrom
sapphi-red:feat/scaffold-with-rolldown-vite
Sep 22, 2025
Merged

feat(create-vite): scaffold with rolldown-vite#20739
sapphi-red merged 8 commits into
vitejs:mainfrom
sapphi-red:feat/scaffold-with-rolldown-vite

Conversation

@sapphi-red
Copy link
Copy Markdown
Member

Description

Added a prompt to use rolldown-vite so that the exposure of rolldown-vite increases a bit more.

@sapphi-red sapphi-red added the feat: create-vite create-vite package label Sep 8, 2025
@sapphi-red sapphi-red marked this pull request as ready for review September 13, 2025 07:30
Comment thread packages/create-vite/src/index.ts Outdated
@github-project-automation github-project-automation Bot moved this to Discussing in Team Board Sep 16, 2025
Comment thread packages/create-vite/src/index.ts
Comment thread packages/create-vite/src/index.ts
Copy link
Copy Markdown
Member

@bluwy bluwy left a comment

Choose a reason for hiding this comment

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

Maybe we can also briefly mention at the docs that they can use create-vite to try rolldown too.

if (useRolldownVite) {
// renovate: datasource=npm depName=rolldown-vite
const rolldownViteVersion = '7.1.12'
const pkgVersion = `npm:rolldown-vite@${rolldownViteVersion}`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
const pkgVersion = `npm:rolldown-vite@${rolldownViteVersion}`
const pkgVersion = `npm:rolldown-vite@^${rolldownViteVersion}`

Should we use a caret here so it's not pinned? (I think it works but haven't tested)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll prefer to keep without it as rolldown-vite doesn't necessarily follow semver.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Alright. In that case, maybe the docs should also mention pinning to a version, but I'm happy to handle that separately.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

@sapphi-red sapphi-red Sep 22, 2025

Choose a reason for hiding this comment

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

or do you mean updating the package.json example?

Copy link
Copy Markdown
Member

@bluwy bluwy Sep 22, 2025

Choose a reason for hiding this comment

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

Yeah I mean the docs mentioning "npm:rolldown-vite@latest", so it's also not pinned to a specific version. It could mention that it's recommended to specify a specific version instead of latest.

Comment thread packages/create-vite/src/index.ts
@sapphi-red sapphi-red merged commit b3227d5 into vitejs:main Sep 22, 2025
16 checks passed
@sapphi-red sapphi-red deleted the feat/scaffold-with-rolldown-vite branch September 22, 2025 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat: create-vite create-vite package

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants