Skip to content

Docs/extract i18n duplicate translation - #34669

Open
MeAkib wants to merge 2 commits into
angular:mainfrom
MeAkib:docs/extract-i18n-duplicate-translation
Open

MeAkib wants to merge 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
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.

2 participants