-
Notifications
You must be signed in to change notification settings - Fork 13.1k
LEGO: Pull request from lego/hb_5378966c-b857-470a-8675-daebef4a6da1_20251125204230298 to main #62805
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
base: main
Are you sure you want to change the base?
LEGO: Pull request from lego/hb_5378966c-b857-470a-8675-daebef4a6da1_20251125204230298 to main #62805
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds localized translations for two new TypeScript diagnostic error messages (error codes 1547 and 1548) related to using and await using declarations in switch statement cases. These messages inform developers that such declarations are not allowed in case or default clauses unless they are contained within a block.
- Adds translations for diagnostic messages 1547 and 1548 across 7 languages
- Messages follow the existing LCL (Localization Comment Language) XML structure
- Translations are inserted in alphabetical order alongside existing error messages
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/loc/lcl/rus/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added Russian translations for the two new diagnostic messages |
| src/loc/lcl/ptb/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added Portuguese (Brazil) translations for the two new diagnostic messages |
| src/loc/lcl/kor/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added Korean translations for the two new diagnostic messages |
| src/loc/lcl/fra/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added French translations for the two new diagnostic messages (with quote mark inconsistencies) |
| src/loc/lcl/esn/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added Spanish translations for the two new diagnostic messages |
| src/loc/lcl/deu/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added German translations for the two new diagnostic messages |
| src/loc/lcl/cht/diagnosticMessages/diagnosticMessages.generated.json.lcl | Added Traditional Chinese translations for the two new diagnostic messages |
| <Item ItemId=";await_using_declarations_are_not_allowed_in_case_or_default_clauses_unless_contained_within_a_block_1548" ItemType="0" PsrId="306" Leaf="true"> | ||
| <Str Cat="Text"> | ||
| <Val><![CDATA['await using' declarations are not allowed in 'case' or 'default' clauses unless contained within a block.]]></Val> | ||
| <Tgt Cat="Text" Stat="Loc" Orig="New"> | ||
| <Val><![CDATA[Объявления "await using" не разрешены в предложениях "case" или "default", если они не находятся внутри блока.]]></Val> | ||
| </Tgt> | ||
| </Str> | ||
| <Disp Icon="Str" /> | ||
| </Item> |
Copilot
AI
Nov 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR only includes localizations for 7 out of 13 supported languages. The following language files are missing the new diagnostic messages for error codes 1547 and 1548:
src/loc/lcl/chs/diagnosticMessages/diagnosticMessages.generated.json.lcl(Chinese Simplified)src/loc/lcl/csy/diagnosticMessages/diagnosticMessages.generated.json.lcl(Czech)src/loc/lcl/ita/diagnosticMessages/diagnosticMessages.generated.json.lcl(Italian)src/loc/lcl/jpn/diagnosticMessages/diagnosticMessages.generated.json.lcl(Japanese)src/loc/lcl/plk/diagnosticMessages/diagnosticMessages.generated.json.lcl(Polish)src/loc/lcl/trk/diagnosticMessages/diagnosticMessages.generated.json.lcl(Turkish)
These files should be updated to include the same two new diagnostic messages:
await_using_declarations_are_not_allowed_in_case_or_default_clauses_unless_contained_within_a_block_1548using_declarations_are_not_allowed_in_case_or_default_clauses_unless_contained_within_a_block_1547
The localized entries should follow the same XML structure and placement as seen in the already-updated language files.
| <Str Cat="Text"> | ||
| <Val><![CDATA['await using' declarations are not allowed in 'case' or 'default' clauses unless contained within a block.]]></Val> | ||
| <Tgt Cat="Text" Stat="Loc" Orig="New"> | ||
| <Val><![CDATA[Les déclarations 'await using' ne sont pas autorisées dans les clauses 'case' ou 'default', sauf si elles sont contenues dans un bloc.]]></Val> |
Copilot
AI
Nov 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The French translation is inconsistent with quote marks. The existing translation for error 1546 at line 18640 uses guillemet quotes (« await using »), but the new translation for error 1548 uses regular single quotes ('await using'). For consistency, line 18649 should use:
Les déclarations « await using » ne sont pas autorisées dans les clauses « case » ou « default », sauf si elles sont contenues dans un bloc.
This maintains consistency with the existing French localization style.
| <Str Cat="Text"> | ||
| <Val><![CDATA['using' declarations are not allowed in 'case' or 'default' clauses unless contained within a block.]]></Val> | ||
| <Tgt Cat="Text" Stat="Loc" Orig="New"> | ||
| <Val><![CDATA[Les déclarations 'using' ne sont pas autorisées dans les clauses 'case' ou 'default' sauf si elles sont incluses dans un bloc.]]></Val> |
Copilot
AI
Nov 25, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The French translation is inconsistent with quote marks. The existing translation for error 1545 at line 19513 uses guillemet quotes (« using »), but the new translation for error 1547 uses regular single quotes ('using'). For consistency, line 19522 should use:
Les déclarations « using » ne sont pas autorisées dans les clauses « case » ou « default » sauf si elles sont incluses dans un bloc.
This maintains consistency with the existing French localization style.
LEGO: Pull request from lego/hb_5378966c-b857-470a-8675-daebef4a6da1_20251125204230298 to main with localized lcls