Skip to content

fix(reporter)!: remove deprecated APIs - #8223

Merged
sheremet-va merged 7 commits into
vitest-dev:mainfrom
AriPerkkio:fix/deprecated-on-finish
Aug 4, 2025
Merged

sheremet-va merged 7 commits into
vitest-dev:mainfrom
AriPerkkio:fix/deprecated-on-finish

Conversation

@AriPerkkio

Copy link
Copy Markdown
Member

Description

  • Ref. feat: introduce the new reporter API #7069
  • fix(reporter)!: remove onFinished
    • Simply using testModule.task everywhere. At first I started to convert all task usage to testModule, but it was too much work. Got BaseReporter quite far but error handling was complex now.
  • fix(reporter)!: remove onPathsCollected
  • fix(reporter)!: remove onSpecsCollected
  • fix(reporter)!: remove onCollected
  • fix(reporter)!: mark onTaskUpdate as @internal
    • Marking as internal instead of removing due to WebSocketReporter, UI and BenchmarkReporter usage

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

  • It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
  • Ideally, include a test that fails without this PR but passes with it.
  • Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

  • Run the tests with pnpm test:ci.

Documentation

  • If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

  • Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

@AriPerkkio AriPerkkio added this to the 4.0.0 milestone Jun 26, 2025
@AriPerkkio AriPerkkio changed the title feat(reporter)!: remove deprecated APIs fix(reporter)!: remove deprecated APIs Jun 26, 2025
@netlify

netlify Bot commented Jun 26, 2025 •

Copy link
Copy Markdown

✅ Deploy Preview for vitest-dev ready!

Name Link
🔨 Latest commit 8a6cce0
🔍 Latest deploy log https://app.netlify.com/projects/vitest-dev/deploys/68909a1d9b375f000891db94
😎 Deploy Preview https://deploy-preview-8223--vitest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@AriPerkkio
AriPerkkio force-pushed the fix/deprecated-on-finish branch from 3c747d6 to 2d0aaef Compare June 26, 2025 05:42
Comment thread packages/vitest/src/api/setup.ts Outdated
@AriPerkkio
AriPerkkio force-pushed the fix/deprecated-on-finish branch from 2d0aaef to 6425da0 Compare June 26, 2025 18:00
@AriPerkkio
AriPerkkio marked this pull request as ready for review June 26, 2025 18:40
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.

2 participants