Skip to content
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

fix(react): respect unitTestRunner passed to the generator #23383

Merged
merged 1 commit into from
May 16, 2024

Conversation

ndcunningham
Copy link
Contributor

closes: #22276

@ndcunningham ndcunningham added the scope: react Issues related to React support for Nx label May 14, 2024
@ndcunningham ndcunningham self-assigned this May 14, 2024
@ndcunningham ndcunningham requested a review from a team as a code owner May 14, 2024 19:45
@ndcunningham ndcunningham requested a review from xiongemi May 14, 2024 19:45
Copy link

vercel bot commented May 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 16, 2024 8:19pm

@ndcunningham ndcunningham force-pushed the fix/react-vite-jest branch 3 times, most recently from 9f3c329 to 497341d Compare May 15, 2024 18:40
@ndcunningham ndcunningham requested a review from a team as a code owner May 15, 2024 18:40
@@ -163,7 +163,6 @@ describe('nx init (for React - legacy)', () => {
);

const packageJson = readJson('package.json');
expect(packageJson.devDependencies['@nx/jest']).toBeUndefined();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

jest is the default so if the unitTestRunner is not set "none" this will be used.

@ndcunningham ndcunningham force-pushed the fix/react-vite-jest branch from 497341d to 10c6439 Compare May 15, 2024 20:04
@ndcunningham ndcunningham requested a review from a team as a code owner May 15, 2024 20:04
@ndcunningham ndcunningham requested a review from mandarini May 15, 2024 20:04
@ndcunningham ndcunningham force-pushed the fix/react-vite-jest branch 3 times, most recently from 1a9aaf6 to fcd0e2b Compare May 16, 2024 03:10
@ndcunningham ndcunningham force-pushed the fix/react-vite-jest branch from fcd0e2b to 038bf3e Compare May 16, 2024 20:17
@ndcunningham ndcunningham merged commit 2e63056 into master May 16, 2024
6 checks passed
@ndcunningham ndcunningham deleted the fix/react-vite-jest branch May 16, 2024 20:59
FrozenPandaz pushed a commit that referenced this pull request May 21, 2024
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope: react Issues related to React support for Nx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@nx/react:application generator - cannot have unitTestRunner jest with bundler vite
3 participants