Skip to content

fix(angular): respect skipTsConfig option in library generator #31738

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 26, 2025

Conversation

Coly010
Copy link
Contributor

@Coly010 Coly010 commented Jun 26, 2025

Current Behavior

The Angular library generator ignores the skipTsConfig option and always modifies the tsconfig.base.json file, even when users explicitly set skipTsConfig=true to avoid TypeScript configuration changes.

Expected Behavior

When skipTsConfig=true is passed to the Angular library generator, the tsconfig.base.json file should not be modified. Project-specific tsconfig files should still be created as expected, but the workspace-level TypeScript
configuration should remain untouched.

Related Issue(s)

Fixes #31185

Fixed issue where the Angular library generator ignored the skipTsConfig option.
When skipTsConfig=true, the tsconfig.base.json file is no longer modified.
Project-specific tsconfig files are still created as expected.

Closes #31185
@Coly010 Coly010 requested a review from a team as a code owner June 26, 2025 10:38
@Coly010 Coly010 self-assigned this Jun 26, 2025
@Coly010 Coly010 requested a review from jaysoo June 26, 2025 10:38
Copy link

vercel bot commented Jun 26, 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 26, 2025 10:51am

Copy link

nx-cloud bot commented Jun 26, 2025

View your CI Pipeline Execution ↗ for commit aff479a.

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

☁️ Nx Cloud last updated this comment at 2025-06-26 11:09:39 UTC

@Coly010 Coly010 merged commit babf86a into master Jun 26, 2025
8 checks passed
@Coly010 Coly010 deleted the 311850-skip-ts-config branch June 26, 2025 11:18
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.

Angular library generator ignores option skipTsConfig
2 participants