-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: allow inline workspace configuration #6923
feat: allow inline workspace configuration #6923
Conversation
✅ Deploy Preview for vitest-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
355dcb4
to
851b02f
Compare
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.
Looks good, this will be really handy!
In some comments the extends: true
says it makes project inherit all options from root even when it inherits only the ones that can be used in project level settings. Maybe that could be made more specific. 🤔
I don't understand what you mean. |
Oh that's right! I was thinking this the other way around. And now that |
d8aa6e5
Description
Fixes #6913
This PR allows specifying an array in
test.workspaces
option. This will override thevitest.workspace
file if there is one: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:
.