Skip to content

Conversation

@sheremet-va
Copy link
Member

@sheremet-va sheremet-va commented Mar 6, 2025

Description

This PR:

  • Moves vitest/browser to vitest/internal/browser
  • Moves used exports from vitest/utils to vitest/internal/browser entry point
  • Removes vitest/utils entry point
  • Removes undocumented API: isFirstRun, runOnce, getRunningMode, isWatchMode

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:.

@sheremet-va sheremet-va requested a review from AriPerkkio March 6, 2025 15:23
@netlify
Copy link

netlify bot commented Mar 6, 2025

Deploy Preview for vitest-dev ready!

Name Link
🔨 Latest commit f99945e
🔍 Latest deploy log https://app.netlify.com/projects/vitest-dev/deploys/6829e226848f730008a62567
😎 Deploy Preview https://deploy-preview-7618--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.

@sheremet-va sheremet-va added the p2-to-be-discussed Enhancement under consideration (priority) label Mar 31, 2025
@sheremet-va sheremet-va moved this to P2 - 5 in Team Board Mar 31, 2025
@sheremet-va sheremet-va moved this from P2 - 5 to Approved in Team Board May 1, 2025
@sheremet-va sheremet-va changed the title test: add test to monitor vitest exports fix: remove unused exports May 18, 2025
@sheremet-va sheremet-va merged commit 33d0538 into vitest-dev:main May 19, 2025
12 of 13 checks passed
@sheremet-va sheremet-va deleted the test/exports-test branch May 19, 2025 12:36
Comment on lines -77 to -80
"./utils": {
"types": "./dist/utils.d.ts",
"default": "./dist/utils.js"
},

Choose a reason for hiding this comment

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

This is a breaking change. This may or may not be intentional, but I want to make sure you’re aware. I had to make code changes because of this in remcohaszing/monaco-languageserver-types@7183fd5.

Copy link
Member Author

Choose a reason for hiding this comment

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

It was never documented, so we don't consider it a breaking change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p2-to-be-discussed Enhancement under consideration (priority)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants