fix: drop vite 8.beta support#9862
Conversation
|
This looks good to me, but I am not so sure it addresses the issue in yarn. vitest/mocker for example had 7.0.0-0 up until the new 4.1 version: https://npmx.dev/package-code/@vitest/mocker/v/4.0.18/package.json which means yarn supports this syntax |
@vitest/browser
@vitest/browser-playwright
@vitest/browser-preview
@vitest/browser-webdriverio
@vitest/coverage-istanbul
@vitest/coverage-v8
@vitest/expect
@vitest/mocker
@vitest/pretty-format
@vitest/runner
@vitest/snapshot
@vitest/spy
@vitest/ui
@vitest/utils
vitest
@vitest/web-worker
@vitest/ws-client
commit: |
But not as For example, this one works fine on {
"name": "yarn-1",
"packageManager": "yarn@1.22.22",
"dependencies": {
"vite": "^6.0.0",
"vitest": "3.2.0"
}
}
|
|
Hey guys, this PR does not solve the #9859 issue. Example: {
"name": "foo",
"version": "1.0.0",
"dependencies": {
"vite": "^7.0.0",
"vitest": "https://pkg.pr.new/vitest@b78f538"
}
}Then It seems that defining |
I think the second installation just always succeeds? The error sometimes appear for me, sometimes not. Maybe it caches something? This seems like an issue with Yarn 1 rather than with Vitest's package.json. |
Not really, at least in my env. I checked |
Does it work if you delete node_modules? |
Unfortunately not. I checked it with only the |
Description
Resolves #9859
Vite 8.0.0 is now
latestso let's drop the beta support as it was planned. Supporting Vite's beta version was always meant to be experimental, so no breaking changes here.Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
pnpm-lock.yamlunless you introduce a new test example.Tests
pnpm test:ci.Documentation
pnpm run docscommand.Changesets
feat:,fix:,perf:,docs:, orchore:.