Skip to content

fix(js): update outDir in runtime tsconfig files to match types export in package.json#30217

Merged
jaysoo merged 1 commit intomasterfrom
misc/fix-typecheck-task
Feb 28, 2025
Merged

fix(js): update outDir in runtime tsconfig files to match types export in package.json#30217
jaysoo merged 1 commit intomasterfrom
misc/fix-typecheck-task

Conversation

@leosvelperez
Copy link
Copy Markdown
Member

Current Behavior

In the TS solution setup, several project generators produce the runtime tsconfig files (e.g. tsconfig.lib.json) with the outDir set to out-tsc/<project name>. This causes issues with the inferred typecheck task because the project package.json has the types export pointing to dist/..., which wouldn't be produced by typecheck.

Expected Behavior

In the TS solution setup, project generators should produce the runtime tsconfig files (e.g. tsconfig.lib.json) with the outDir set to a path (dist) that matches the value in the project package.json's types export.

Related Issue(s)

Fixes #

@leosvelperez leosvelperez self-assigned this Feb 28, 2025
@leosvelperez leosvelperez requested review from a team and Coly010 as code owners February 28, 2025 11:48
@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 28, 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) Feb 28, 2025 11:48am

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Feb 28, 2025

View your CI Pipeline Execution ↗ for commit 93096a3.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 41m 5s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 16s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx format:check --base= --he... ✅ Succeeded 7s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 5s View ↗
nx documentation ✅ Succeeded 1m 32s View ↗

☁️ Nx Cloud last updated this comment at 2025-02-28 12:35:07 UTC

@jaysoo jaysoo merged commit a5d2003 into master Feb 28, 2025
@jaysoo jaysoo deleted the misc/fix-typecheck-task branch February 28, 2025 21:11
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 6, 2025

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 Mar 6, 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.

3 participants