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

feat(testing): disable jest runtime by default when inferring targets #29917

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

jaysoo
Copy link
Member

@jaysoo jaysoo commented Feb 6, 2025

This PR updates @nx/jest/plugin such that disableJestRuntime option is true by default. Users will need to set it to false to bring in jest-config and jest-runtime to compute atomized targets. We're leaving it as an option if anyone runs into discrepancies between our calculation and what jest-runtime calculates for test files within a project.

Current Behavior

Jest runtime is used by default and is potentially slow if you use many transforms, presets, etc. in the jest config.

Expected Behavior

Jest runtime is not used by default, and users have to option of enabling it.

Related Issue(s)

Fixes #

@jaysoo jaysoo requested a review from a team as a code owner February 6, 2025 19:54
@jaysoo jaysoo requested a review from AgentEnder February 6, 2025 19:54
Copy link

vercel bot commented Feb 6, 2025

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

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Feb 7, 2025 4:52pm

Copy link

nx-cloud bot commented Feb 6, 2025

View your CI Pipeline Execution ↗ for commit 81bfeeb.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 4m 29s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 24s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check --base= --he... ✅ Succeeded 11s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 10s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-02-07 17:31:41 UTC

@jaysoo jaysoo force-pushed the feat/disable-jest-runtime branch 4 times, most recently from 5bf8645 to f00e59c Compare February 7, 2025 16:39
@jaysoo jaysoo force-pushed the feat/disable-jest-runtime branch from f00e59c to f1f2000 Compare February 7, 2025 16:47
@jaysoo jaysoo force-pushed the feat/disable-jest-runtime branch from f1f2000 to 81bfeeb Compare February 7, 2025 16:48
@FrozenPandaz FrozenPandaz merged commit 9bc6317 into master Feb 7, 2025
12 checks passed
@FrozenPandaz FrozenPandaz deleted the feat/disable-jest-runtime branch February 7, 2025 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants