-
Notifications
You must be signed in to change notification settings - Fork 1k
Add support for Vitest 3.1.x #8804
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
Conversation
🦋 Changeset detectedLatest commit: 7469234 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Hey @petebacondarwin 👋 could you please review this dependency update? 🙇♂️ |
|
Running the tests but this PR will need a bit more: a changeset and possible updates to the version of Vitest we are using in our testing and fixtures... |
|
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-wrangler-8804You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/8804/npm-package-wrangler-8804Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-wrangler-8804 dev path/to/script.jsAdditional artifacts:cloudflare-workers-bindings-extension: wget https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-workers-bindings-extension-8804 -O ./cloudflare-workers-bindings-extension.0.0.0-v3b6328a53.vsix && code --install-extension ./cloudflare-workers-bindings-extension.0.0.0-v3b6328a53.vsixcreate-cloudflare: npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-create-cloudflare-8804 --no-auto-update@cloudflare/kv-asset-handler: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-kv-asset-handler-8804miniflare: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-miniflare-8804@cloudflare/pages-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-pages-shared-8804@cloudflare/unenv-preset: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-unenv-preset-8804@cloudflare/vite-plugin: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-vite-plugin-8804@cloudflare/vitest-pool-workers: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-vitest-pool-workers-8804@cloudflare/workers-editor-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-workers-editor-shared-8804@cloudflare/workers-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-workers-shared-8804@cloudflare/workflows-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14359986022/npm-package-cloudflare-workflows-shared-8804Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
Thanks @petebacondarwin I've added |
penalosa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks generally good, but shouldn't update esbuild
pnpm-workspace.yaml
Outdated
| vitest: "~3.1.1" | ||
| vite: "^5.4.14" | ||
| "ws": "8.18.0" | ||
| esbuild: "0.24.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR shouldn't be updating esbuild—it should be limited to Vitest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha, I've updated esbuild due to vulnerability concern.
Anyhow, I've dropped it for now.
|
Updated, thanks for the support @petebacondarwin Back to you 🏓 |
|
Looks like Vitest 3.1 is formatting snapshots slightly differently? I'll update and push a commit. |
|
@petebacondarwin I just solved the conflicts, back to you 🏓 |
alsuren
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume D1 has been tagged for a code owner review because of packages/wrangler/src/tests/d1/insights.test.ts , so I'm happy to approve that bit to unblock you (assuming that tests still pass), but please wait for someone on the wrangler team to approve the rest before merging.
|
A Wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-wrangler-8804Prereleases for other packages:
wget https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-workers-bindings-extension-8804 -O ./cloudflare-workers-bindings-extension.0.0.0-v7342524d4.vsix && code --install-extension ./cloudflare-workers-bindings-extension.0.0.0-v7342524d4.vsix
npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-create-cloudflare-8804 --no-auto-update
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-kv-asset-handler-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-miniflare-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-pages-shared-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-unenv-preset-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-vite-plugin-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-vitest-pool-workers-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-workers-editor-shared-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-workers-shared-8804
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/14439012554/npm-package-cloudflare-workflows-shared-8804Note that these links will no longer work once the GitHub Actions artifact expires. |
|
Congratulations @boimw, the maintainer of this repository has issued you a holobyte! Here it is: https://holopin.io/holobyte/cm9j29xnm00470cjxsigjzyva This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
Fixes #8743.
This MR adds support for Vitest 3.1.x and it fixes peer dependency with
vitest,@vitest/coverage-istanbuland similar packages.