Skip to content

LEGO: Pull request from lego/hb_5378966c-b857-470a-8675-daebef4a6da1_20250702194908541 to main #61992

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

Conversation

csigs
Copy link
Member

@csigs csigs commented Jul 2, 2025

LEGO: Pull request from lego/hb_5378966c-b857-470a-8675-daebef4a6da1_20250702194908541 to main with localized lcls

@Copilot Copilot AI review requested due to automatic review settings July 2, 2025 19:49
@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Jul 2, 2025
@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Jul 2, 2025
Copy link

@Copilot Copilot AI left a 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 updates localized diagnostic messages by standardizing the checkJS option casing to checkJs across language files and adds a new diagnostic entry for calls without signatures.

  • Normalize the option name from checkJS to checkJs in source values and IDs
  • Introduce a new message Call target does not contain any signatures with translations
  • Apply these changes across all supported locale files

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/loc/lcl/trk/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/rus/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/ptb/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/plk/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/kor/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/jpn/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/ita/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/fra/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/esn/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/deu/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/csy/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/cht/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message
src/loc/lcl/chs/diagnosticMessages/diagnosticMessages.generated.json.lcl Correct checkJs casing and add missing-signature message

Comment on lines 1886 to 1888
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Zezwalaj plikom JavaScript na udział w programie. Użyj opcji „checkJS”, aby uzyskać błędy z tych plików.]]></Val>
</Tgt>
Copy link
Preview

Copilot AI Jul 2, 2025

Choose a reason for hiding this comment

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

Update the Polish translation to use “checkJs” (lowercase 's') to match the updated option casing.

Suggested change
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Zezwalaj plikom JavaScript na udział w programie. Użyj opcji „checkJS”, aby uzyskać błędy z tych plików.]]></Val>
</Tgt>
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Zezwalaj plikom JavaScript na udział w programie. Użyj opcji „checkJs”, aby uzyskać błędy z tych plików.]]></Val>
</Tgt>

Copilot uses AI. Check for mistakes.

<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Autorisez les fichiers JavaScript à faire partie de votre programme. Utilisez l’option « checkJS » pour obtenir des erreurs à partir de ces fichiers.]]></Val>
<Val><![CDATA[Autorisez les fichiers JavaScript à faire partie de votre programme. Utilisez l’option « checkJS » pour obtenir des erreurs à partir de ces fichiers.]]></Val>
Copy link
Preview

Copilot AI Jul 2, 2025

Choose a reason for hiding this comment

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

Update the French translation to use « checkJs » (lowercase 's') to match the updated option casing.

Suggested change
<Val><![CDATA[Autorisez les fichiers JavaScript à faire partie de votre programme. Utilisez l’option « checkJS » pour obtenir des erreurs à partir de ces fichiers.]]></Val>
<Val><![CDATA[Autorisez les fichiers JavaScript à faire partie de votre programme. Utilisez l’option « checkJs » pour obtenir des erreurs à partir de ces fichiers.]]></Val>

Copilot uses AI. Check for mistakes.

Comment on lines 1908 to 1910
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Permita que los archivos JavaScript formen parte del programa. Use la opción "checkJS" para obtener errores de estos archivos.]]></Val>
<Val><![CDATA[Permitir que los archivos JavaScript formen parte del programa. Use la opción "checkJS" para obtener errores de estos archivos.]]></Val>
</Tgt>
Copy link
Preview

Copilot AI Jul 2, 2025

Choose a reason for hiding this comment

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

Update the Spanish translation to use "checkJs" (lowercase 's') to match the updated option casing.

Copilot uses AI. Check for mistakes.

Comment on lines 1893 to 1895
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Lassen Sie zu, dass JavaScript-Dateien Teil Ihres Programms werden. Verwenden Sie die Option „checkJS“, um Fehler aus diesen Dateien abzurufen.]]></Val>
</Tgt>
Copy link
Preview

Copilot AI Jul 2, 2025

Choose a reason for hiding this comment

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

Update the German translation to use „checkJs“ (lowercase 's') to match the updated option casing.

Suggested change
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Lassen Sie zu, dass JavaScript-Dateien Teil Ihres Programms werden. Verwenden Sie die Option „checkJS“, um Fehler aus diesen Dateien abzurufen.]]></Val>
</Tgt>
<Tgt Cat="Text" Stat="Loc" Orig="New">
<Val><![CDATA[Lassen Sie zu, dass JavaScript-Dateien Teil Ihres Programms werden. Verwenden Sie die Option „checkJs“, um Fehler aus diesen Dateien abzurufen.]]></Val>
</Tgt>

Copilot uses AI. Check for mistakes.

@github-project-automation github-project-automation bot moved this from Not started to Needs merge in PR Backlog Jul 2, 2025
@jakebailey jakebailey merged commit 0018c9f into main Jul 2, 2025
32 checks passed
@jakebailey jakebailey deleted the lego/hb_5378966c-b857-470a-8675-daebef4a6da1_20250702194908541 branch July 2, 2025 21:35
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants