Skip to content

Conversation

@lumirlumir
Copy link
Member

@lumirlumir lumirlumir commented Nov 4, 2025

Prerequisites checklist

What is the purpose of this pull request?

In this PR, I've migrated vitest and @vitest/coverage-v8 to v4.

This change have two related breaking changes.

  • First: toMatchFileSnapshot() must now be awaited.
스크린샷 2025-11-04 175849
  • Second: Require Node.js 20 or newer.

Ref: vitest-dev/vitest#8608

So, I've updated them accordingly.

What changes did you make? (Give an overview)

In this PR, I've migrated vitest and @vitest/coverage-v8 to v4.

Related Issues

Ref: #219

Is there anything you'd like reviewers to focus on?

Everything is ready for review. However, vitest and @vitest/coverage-v8 require Node.js 20 or newer (they use node:inspector internally), I'm marking this PR as a draft until #206 is merged. (Ref: vitest-dev/vitest#8608)

image

@lumirlumir lumirlumir changed the title test: migrate vite and @vitest/coverage-v8 to v4 test: migrate vitest and @vitest/coverage-v8 to v4 Nov 4, 2025
@lumirlumir lumirlumir moved this from Needs Triage to Blocked in Triage Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Blocked

Development

Successfully merging this pull request may close these issues.

2 participants