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(misc): use @swc/jest instead of ts-jest for the ts solution setup #29718

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

leosvelperez
Copy link
Member

@leosvelperez leosvelperez commented Jan 22, 2025

Current Behavior

When using the TS solution setup and jest is used, ts-jest is used as the transformer in most cases (except when the build compiler is swc). The ts-jest transformer doesn't support modern module resolutions like nodenext and it doesn't support TS project references either.

Expected Behavior

When using the TS solution setup and jest is used, @swc/jest should be used as the transformer in cases where previously ts-jest was being used and regardless of using swc as the build compiler.

Related Issue(s)

Fixes #

@leosvelperez leosvelperez self-assigned this Jan 22, 2025
Copy link

vercel bot commented Jan 22, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jan 23, 2025 2:12pm

Copy link

nx-cloud bot commented Jan 22, 2025

View your CI Pipeline Execution ↗ for commit 8517582.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 7m 31s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 28s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check --base=a0cfe... ✅ Succeeded 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation --no-dte ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-23 15:05:19 UTC

@leosvelperez leosvelperez force-pushed the js/ts-solution-use-swc-jest branch 2 times, most recently from 5546e48 to f8020ea Compare January 23, 2025 11:22
@leosvelperez leosvelperez marked this pull request as ready for review January 23, 2025 12:12
@leosvelperez leosvelperez requested review from a team and Coly010 as code owners January 23, 2025 12:12
@leosvelperez leosvelperez force-pushed the js/ts-solution-use-swc-jest branch from f8020ea to 8517582 Compare January 23, 2025 14:02
@jaysoo jaysoo merged commit d601561 into master Jan 23, 2025
6 checks passed
@jaysoo jaysoo deleted the js/ts-solution-use-swc-jest branch January 23, 2025 16:59
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.

3 participants