Skip to content

chore: upgrade vite-plus to 0.1.20-alpha.4#1

Draft
fengmk2 wants to merge 1 commit into
mainfrom
update-vite-plus-alpha-0.1.20-alpha.4
Draft

chore: upgrade vite-plus to 0.1.20-alpha.4#1
fengmk2 wants to merge 1 commit into
mainfrom
update-vite-plus-alpha-0.1.20-alpha.4

Conversation

@fengmk2
Copy link
Copy Markdown
Owner

@fengmk2 fengmk2 commented Apr 28, 2026

Upgrade vite-plus and related packages to 0.1.20-alpha.4 alpha version.

@fengmk2 fengmk2 self-assigned this Apr 28, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the project's core build and test dependencies, specifically moving vite, vite-plus, and vitest to version 0.1.20-alpha.4. It also includes updates for oxfmt and oxlint and removes several unused WASM-related bindings. A critical feedback point notes that the new version of vitest requires @vitest/coverage-v8 version 4.1.5, which currently conflicts with the version specified in the project's dependencies.

Comment thread package.json
"vitest": "npm:@voidzero-dev/vite-plus-test@^0.1.14"
"vite": "npm:@voidzero-dev/vite-plus-core@0.1.20-alpha.4",
"vite-plus": "0.1.20-alpha.4",
"vitest": "npm:@voidzero-dev/vite-plus-test@0.1.20-alpha.4"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Upgrading vitest to npm:@voidzero-dev/vite-plus-test@0.1.20-alpha.4 introduces a new peer dependency requirement for @vitest/coverage-v8 at version 4.1.5. The version currently specified in devDependencies is ^4.1.2, which will resolve to 4.1.2 and not satisfy the new requirement. Since your vite.config.ts specifies v8 as the coverage provider, this mismatch could lead to issues. Please update @vitest/coverage-v8 to ^4.1.5 to ensure compatibility.

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.

1 participant