Skip to content

fix(core): use ts-node option from tsconfig files when creating transpiler #31469

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

Merged
merged 1 commit into from
Jun 5, 2025

Conversation

leosvelperez
Copy link
Member

Current Behavior

When creating a ts-node transpiler, only compilerOptions are provided. Because we instruct ts-node to skip reading the tsconfig (this was previously done to avoid some edge cases), other options in the tsconfig files are lost (e.g. ts-node specific options).

This was previously reported at #21695 and fixed by #21723, but a rework at a later point caused a regression.

Expected Behavior

When creating a ts-node transpiler, we should provide compilerOptions and the ts-node options.

@leosvelperez leosvelperez self-assigned this Jun 5, 2025
@leosvelperez leosvelperez requested a review from a team as a code owner June 5, 2025 12:19
@leosvelperez leosvelperez requested a review from Cammisuli June 5, 2025 12:19
Copy link

vercel bot commented Jun 5, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jun 5, 2025 0:54am

Copy link
Contributor

nx-cloud bot commented Jun 5, 2025

View your CI Pipeline Execution ↗ for commit 58cdfeb.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 8m 1s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 21s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 2s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-06-05 13:25:41 UTC

@FrozenPandaz FrozenPandaz merged commit 9323403 into master Jun 5, 2025
9 of 10 checks passed
@FrozenPandaz FrozenPandaz deleted the core/fix-ts-node-transpiler-options branch June 5, 2025 18:04
FrozenPandaz pushed a commit that referenced this pull request Jun 5, 2025
…nspiler (#31469)

## Current Behavior

When creating a `ts-node` transpiler, only `compilerOptions` are
provided. Because we instruct `ts-node` to skip reading the tsconfig
(this was previously done to avoid some edge cases), other options in
the tsconfig files are lost (e.g. `ts-node` specific options).

This was previously reported at #21695
and fixed by #21723, but a rework at a
later point caused a regression.

## Expected Behavior

When creating a `ts-node` transpiler, we should provide
`compilerOptions` and the `ts-node` options.

(cherry picked from commit 9323403)
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 Jun 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy