Skip to content

docs(@angular/build): note tsconfig fallback for library declarationMap and compilationMode - #34483

Merged
clydin merged 2 commits into
angular:mainfrom
MeAkib:docs/library-tsconfig-option-defaults
Oct 22, 2026
Merged

clydin merged 2 commits into
angular:mainfrom
MeAkib:docs/library-tsconfig-option-defaults

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

The library builder no longer sets schema defaults for declarationMap and compilationMode, so the values from the TypeScript configuration apply when the options are not set. The option descriptions did not mention this. Document the fallback, including the partial default for compilationMode.

…Map` and `compilationMode`

The library builder no longer sets schema defaults for `declarationMap` and `compilationMode`, so
the values from the TypeScript configuration apply when the options are not set. The option
descriptions did not mention this. Document the fallback, including the `partial` default for
`compilationMode`.
@angular-robot angular-robot Bot added area: docs Related to the documentation area: @angular/build labels Oct 22, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the schema descriptions for declarationMap and compilationMode in the Angular library builder to clarify their default behaviors based on the TypeScript configuration file. The review feedback suggests improving the description of declarationMap by specifying the precise path compilerOptions.declarationMap rather than just declarationMap to maintain consistency and accuracy.

Comment thread packages/angular/build/src/builders/library/schema.json Outdated
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Oct 22, 2026
@clydin
clydin merged commit df1c78d into angular:main Oct 22, 2026
41 checks passed
@clydin

clydin commented Oct 22, 2026

Copy link
Copy Markdown
Member

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build area: docs Related to the documentation target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants