Skip to content

Docs/extract i18n duplicate translation - #34669

Merged
clydin merged 2 commits into
angular:mainfrom
MeAkib:docs/extract-i18n-duplicate-translation
Nov 11, 2026
Merged

clydin merged 2 commits into
angular:mainfrom
MeAkib:docs/extract-i18n-duplicate-translation

Conversation

@MeAkib

@MeAkib MeAkib commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@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 i18nDuplicateTranslation configuration in both packages/angular and packages/angular_devkit by refining the description and setting a default value of 'warning'. I have no feedback to provide as there were no review comments.

…ate message option

The `i18nDuplicateTranslation` option of the extract-i18n builder controls how duplicate messages
found during extraction are reported, defaulting to a warning when the option is not set. The schema
description is copied from the build-time option that handles duplicate translations when merging
translation files, and no default is documented. Describe the extraction behavior and document the
`warning` default that the option normalization applies.
…act-i18n duplicate message option

The `i18nDuplicateTranslation` option of the extract-i18n builder controls how duplicate messages
found during extraction are reported, defaulting to a warning when the option is not set. The schema
description is copied from the build-time option that handles duplicate translations when merging
translation files, and no default is documented. Describe the extraction behavior and document the
`warning` default that the option normalization applies.
@alan-agius4
alan-agius4 force-pushed the docs/extract-i18n-duplicate-translation branch from bb67f7e to 487966c Compare November 9, 2026 10:58
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Nov 9, 2026
@clydin
clydin merged commit 2bac6c2 into angular:main Nov 11, 2026
41 checks passed
@clydin

clydin commented Nov 11, 2026

Copy link
Copy Markdown
Member

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

@MeAkib
MeAkib deleted the docs/extract-i18n-duplicate-translation branch November 11, 2026 19:54
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: @angular-devkit/build-angular area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants