Repository navigation
[localization] Include template JSON files in OneLocBuild configuration. Fixes #20717 - #25717
Conversation
The templatestrings.en.json files in dotnet/Templates were not included in the LocProject.json configuration, which meant OneLocBuild never translated them. This caused Polish and Chinese Traditional translations to remain outdated (still referencing '.NET 6'). Replace the static LocProject.json.in template with a generate-loc-project.sh script that dynamically discovers all templatestrings.en.json files and includes them in the generated LocProject.json. This also fixes a duplicate entry for Microsoft.Macios.Generator/Resources.resx that was in the original .in file. Fixes #20717 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Updates the localization configuration used by OneLocBuild so that template localization source files (templatestrings.en.json under dotnet/Templates/) are included, fixing missed translations for template strings (issue #20717). It also removes a previously duplicated .resx entry by moving from a static JSON template to a generated file.
Changes:
- Replace the static
LocProject.json.intemplate with a bash generator that discovers alltemplatestrings.en.jsonfiles underdotnet/Templates/. - Update the devops Makefile target to generate
Localize/LocProject.jsonvia the new script. - Remove the duplicate
Microsoft.Macios.Generator/Resources.resxentry by regenerating the list from a single authoritative script.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tools/devops/Makefile | Switches LocProject generation to run the new discovery script. |
| tools/devops/LocProject.json.in | Removes the static template configuration (including a duplicate entry). |
| tools/devops/generate-loc-project.sh | New generator that emits LocProject.json with static .resx items plus discovered template JSON files. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
β [PR Build #39bfb36] Build passed (Detect API changes) βPipeline on Agent |
This comment has been minimized.
This comment has been minimized.
β [PR Build #39bfb36] Build passed (Build packages) βPipeline on Agent |
β API diff for current PR / commitNET (empty diffs)β API diff vs stableNET (empty diffs)βΉοΈ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
β [PR Build #39bfb36] Build passed (Build macOS tests) βPipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
π [CI Build #39bfb36] Test results πTest resultsβ All tests passed on VSTS: test results. π All 220 tests passed π Tests countsβ
assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS testsβ
Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
The
templatestrings.en.jsonfiles indotnet/Templates/were not included in theLocProject.jsonconfiguration, which meant OneLocBuild never picked them up for translation.This PR replaces the static
LocProject.json.intemplate with agenerate-loc-project.shscript that dynamically discovers alltemplatestrings.en.jsonfiles and includes them in the generatedLocProject.json.This also fixes a duplicate entry for
Microsoft.Macios.Generator/Resources.resxthat was in the original.infile.Hopefully fixes #20717
π€ Pull request created by Copilot