-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat(api): startVitest()
to accept stdout
and stdin
#5493
Conversation
✅ Deploy Preview for fastidious-cascaron-4ded94 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
20839b2
to
754dc39
Compare
// TODO: Why should this be picked as well? | ||
// expect(stderr).toContain('FAIL only.test-d.ts') // .only |
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.
Not sure why this started to fail now. The only.test-d.ts
is not defined in the custom TS config.
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 don't remember 🤔
6d65e44
to
7fd7a95
Compare
7fd7a95
to
093f0bd
Compare
startVitest()
to accept stdout
and stdin
startVitest()
to accept stdout
and stdin
Let's merge this as a patch since it modifies the public API which is not affected by the semver. |
Description
Adds new options to
startVitest()
so thatstdout
andstdin
can be handled without overriding globals.Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
pnpm-lock.yaml
unless you introduce a new test example.Tests
pnpm test:ci
.Documentation
pnpm run docs
command.Changesets
feat:
,fix:
,perf:
,docs:
, orchore:
.