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: findRelatedTests flag, to run only changed tests #363
Conversation
✔️ Deploy Preview for vitest-dev ready! 🔨 Explore the source changes: 37e6877 🔍 Inspect the deploy log: https://app.netlify.com/sites/vitest-dev/deploys/61cc8e4ed3293d0009064d22 😎 Browse the preview: https://deploy-preview-363--vitest-dev.netlify.app |
We could use comma and split it ourselves |
Still not ideal. I like space and think we would need it in the future because |
Also I think most of the people using this feature have it configured the same for |
idk, won't that have an ambiguous issue to distinguish from options args or command args? |
In what way? Maybe it's harder for parser, but I think it looks more native to the human eye, no? |
Maybe we should ask in Discord |
vite run basic vite run --global basic vite run --global --repoter dot basic vite run --global --repoter dot --findRelatedTests src/timeout.ts src/mockedA.ts basic |
Should placing options before arguments be allowed tho? Yargs for example can parse array of options I think we should merge this PR as it is and discuss this separately, since it will involve cli package and maybe no one will complain 😄 |
Co-authored-by: Anthony Fu <anthonyfu117@hotmail.com>
Related: #280
CAC allows passing array as an argument only if you repeat option, like:
I personally think jest's way (using space as separator) is more convinient