-
Notifications
You must be signed in to change notification settings - Fork 2.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(vite): allow passing multiple test files to vite:test #17496
feat(vite): allow passing multiple test files to vite:test #17496
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit eac6885. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
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.
Hi thanks for the PR, just a couple things that could be improved!
.../vite/src/migrations/update-16-4-0-update-test-file-config/update-16-4-0-test-file-config.ts
Outdated
Show resolved
Hide resolved
.../src/migrations/update-16-4-0-update-test-file-config/update-16-4-0-test-file-config.spec.ts
Outdated
Show resolved
Hide resolved
.../vite/src/migrations/update-16-4-0-update-test-file-config/update-16-4-0-test-file-config.ts
Outdated
Show resolved
Hide resolved
2729026
to
1beb63e
Compare
1beb63e
to
a877e37
Compare
packages/vite/migrations.json
Outdated
"version": "16.4.0-beta.10", | ||
"description": "Changes the testFile config in the vite:test exectutor from a string to an array of strings", | ||
"cli": "nx", | ||
"implementation": "./src/migrations/update-16-4-0-test-file-config/update-16-4-0-test-file-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.
incorrect path.
a877e37
to
eac6885
Compare
updated the tests to handle the option rename and updated the target for the migration |
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
You can only pass one testfile to run to
vite:test
Expected Behavior
Able to pass multiple test files to run to
vite:test
this is useful for CI for example when using test splitting