Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 3052544 - #10551
Open
dotnet-oneloc-localization[bot] wants to merge 22 commits into
Open
Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 3052544#10551dotnet-oneloc-localization[bot] wants to merge 22 commits into
dotnet-oneloc-localization[bot] wants to merge 22 commits into
Conversation
…: Build ID 3043437
…: Build ID 3043437
…: Build ID 3043437
…: Build ID 3043437
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.
Updates multiple XLIFF resource files with new/updated localized strings for Microsoft Testing Platform, its PackagedApp and Azure DevOps extensions, and MSTest analyzer messages.
Changes:
- Added/updated translations for dynamic extension-related platform messages across several locales.
- Updated analyzer description translations across multiple locales.
- Reset some previously translated extension descriptions/option descriptions back to
state="new"(English source text).
Reviewed changes
Copilot reviewed 52 out of 52 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.zh-Hans.xlf | Adds zh-Hans translations for dynamic extension and command line strings. |
| src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.tr.xlf | Adds Turkish translations for dynamic extension and command line strings. |
| src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.ru.xlf | Adds Russian translations for dynamic extension strings. |
| src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf | Adds Spanish translations for dynamic extension strings. |
| src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf | Adds German translations for dynamic extension strings. |
| src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf | Adds Czech translations for dynamic extension strings. |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.zh-Hant.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.zh-Hans.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.tr.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.ru.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.pt-BR.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.pl.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.ko.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.ja.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.it.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.fr.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.es.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.de.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.cs.xlf | Resets PackagedApp description translation to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.zh-Hant.xlf | Adds some zh-Hant translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.zh-Hans.xlf | Adds some zh-Hans translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.tr.xlf | Adds some Turkish translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.ru.xlf | Adds some Russian translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.pt-BR.xlf | Adds some pt-BR translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.pl.xlf | Adds some Polish translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.ko.xlf | Adds some Korean translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.ja.xlf | Adds some Japanese translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.it.xlf | Adds some Italian translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.fr.xlf | Adds some French translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.es.xlf | Adds some Spanish translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.de.xlf | Adds some German translations; resets one option description to state="new" (English). |
| src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.cs.xlf | Adds some Czech translations; resets one option description to state="new" (English). |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.zh-Hant.xlf | Adds zh-Hant translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.zh-Hans.xlf | Adds zh-Hans translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.tr.xlf | Adds Turkish translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.ru.xlf | Adds Russian translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.pt-BR.xlf | Adds pt-BR translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.pl.xlf | Adds Polish translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.ko.xlf | Adds Korean translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.ja.xlf | Adds Japanese translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.it.xlf | Adds Italian translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.fr.xlf | Adds French translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.es.xlf | Adds Spanish translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.de.xlf | Adds German translations for analyzer descriptions. |
| src/Analyzers/MSTest.Analyzers/xlf/Resources.cs.xlf | Adds Czech translations for analyzer descriptions. |
Suppressed comments (8)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- The German translations appear to swap placeholder meanings compared to the source (e.g.,
{0}is the assembly path in the source but is used as the manifest path in the target). This will render incorrect diagnostic messages at runtime. Please update the target strings so{0},{1},{2}match the same semantics as in the<source>.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - The German translations appear to swap placeholder meanings compared to the source (e.g.,
{0}is the assembly path in the source but is used as the manifest path in the target). This will render incorrect diagnostic messages at runtime. Please update the target strings so{0},{1},{2}match the same semantics as in the<source>.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - The German translations appear to swap placeholder meanings compared to the source (e.g.,
{0}is the assembly path in the source but is used as the manifest path in the target). This will render incorrect diagnostic messages at runtime. Please update the target strings so{0},{1},{2}match the same semantics as in the<source>.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - The Spanish translations contain mismatched/doubled quotes around
{2}(''{2}\"), which will display extra apostrophes and an incorrect closing quote. Replace with consistent quoting (e.g.,... manifiesto de extensión \"{2}\".).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - The Spanish translations contain mismatched/doubled quotes around
{2}(''{2}\"), which will display extra apostrophes and an incorrect closing quote. Replace with consistent quoting (e.g.,... manifiesto de extensión \"{2}\".).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - There is a typo in the placeholder segment (
\"{2}v), and it also looks like the closing quote after{2}is missing. This will produce malformed output; please correct it to keep{2}intact and properly quoted.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf:1 - Missing whitespace after placeholder
{1}({1}deklarovaného). Add a space so the rendered Czech sentence is readable.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.tr.xlf:1 - The Turkish translation uses mixed quote characters around
{0}(‘{0}'). This is likely unintended and will render inconsistently; use consistent quotes (e.g.,'{0}').
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
dotnet-oneloc-localization
Bot
changed the title
dotnet-oneloc-localization
Bot
changed the title
dotnet-oneloc-localization
Bot
changed the title
…: Build ID 3044543
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 52 out of 52 changed files in this pull request and generated 1 comment.
Suppressed comments (11)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- These German targets appear to use the wrong placeholder mapping (e.g.,
{0}is described as the resolved assembly path, but the translation treats{0}as the manifest path and shifts the other placeholders). This will produce misleading diagnostics at runtime. Please adjust the translations so{0}/{1}/{2}correspond to the same values as in the<note>(do not reorder placeholders unless the runtime callsite also changes).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - These German targets appear to use the wrong placeholder mapping (e.g.,
{0}is described as the resolved assembly path, but the translation treats{0}as the manifest path and shifts the other placeholders). This will produce misleading diagnostics at runtime. Please adjust the translations so{0}/{1}/{2}correspond to the same values as in the<note>(do not reorder placeholders unless the runtime callsite also changes).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - These German targets appear to use the wrong placeholder mapping (e.g.,
{0}is described as the resolved assembly path, but the translation treats{0}as the manifest path and shifts the other placeholders). This will produce misleading diagnostics at runtime. Please adjust the translations so{0}/{1}/{2}correspond to the same values as in the<note>(do not reorder placeholders unless the runtime callsite also changes).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets have malformed quoting (
''{2}\",''{3}\",''{0}\", etc.), which will render incorrectly in UI and may break localization tooling expectations. Additionally, line 382 includes an apparent typo (\"{2}v). Please fix the quote pairs to be consistent (e.g.,\"{2}\") and remove the strayvso placeholders render correctly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets have malformed quoting (
''{2}\",''{3}\",''{0}\", etc.), which will render incorrectly in UI and may break localization tooling expectations. Additionally, line 382 includes an apparent typo (\"{2}v). Please fix the quote pairs to be consistent (e.g.,\"{2}\") and remove the strayvso placeholders render correctly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets have malformed quoting (
''{2}\",''{3}\",''{0}\", etc.), which will render incorrectly in UI and may break localization tooling expectations. Additionally, line 382 includes an apparent typo (\"{2}v). Please fix the quote pairs to be consistent (e.g.,\"{2}\") and remove the strayvso placeholders render correctly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets have malformed quoting (
''{2}\",''{3}\",''{0}\", etc.), which will render incorrectly in UI and may break localization tooling expectations. Additionally, line 382 includes an apparent typo (\"{2}v). Please fix the quote pairs to be consistent (e.g.,\"{2}\") and remove the strayvso placeholders render correctly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets have malformed quoting (
''{2}\",''{3}\",''{0}\", etc.), which will render incorrectly in UI and may break localization tooling expectations. Additionally, line 382 includes an apparent typo (\"{2}v). Please fix the quote pairs to be consistent (e.g.,\"{2}\") and remove the strayvso placeholders render correctly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets have malformed quoting (
''{2}\",''{3}\",''{0}\", etc.), which will render incorrectly in UI and may break localization tooling expectations. Additionally, line 382 includes an apparent typo (\"{2}v). Please fix the quote pairs to be consistent (e.g.,\"{2}\") and remove the strayvso placeholders render correctly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf:1 - There is a missing space between
{1}anddeklarovaného, which will concatenate the extension name with the following word at runtime. Insert a space after{1}.
src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.zh-Hans.xlf:1 - This replaces an existing zh-Hans translation with the English source text and marks it as
state=\"new\", which is a user-visible localization regression. If the intent is to update the message content, please update the zh-Hans translation accordingly (and usestate=\"translated\"orneeds-review-translation), rather than resetting the target to English. The same regression pattern appears in multipleExtensionResources.*.xlffiles in this PR.
dotnet-oneloc-localization
Bot
changed the title
…: Build ID 3044575
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 52 out of 52 changed files in this pull request and generated no new comments.
Suppressed comments (12)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- These German translations appear to reorder placeholders incorrectly versus the notes:
{0}is the resolved assembly path,{1}is the extension display name,{2}is the manifest file path. As written, the output will show the wrong values in the wrong places. Please update the translated strings to use the correct placeholder mapping (and keep the same placeholder numbers as the source).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - These German translations appear to reorder placeholders incorrectly versus the notes:
{0}is the resolved assembly path,{1}is the extension display name,{2}is the manifest file path. As written, the output will show the wrong values in the wrong places. Please update the translated strings to use the correct placeholder mapping (and keep the same placeholder numbers as the source).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - These German translations appear to reorder placeholders incorrectly versus the notes:
{0}is the resolved assembly path,{1}is the extension display name,{2}is the manifest file path. As written, the output will show the wrong values in the wrong places. Please update the translated strings to use the correct placeholder mapping (and keep the same placeholder numbers as the source).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets include mismatched/doubled quotes (for example
''{2}\") which will render incorrectly in user-facing messages. Please normalize quoting around placeholders (for example, use\"{2}\"consistently) and ensure opening/closing quotes match in each string.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets include mismatched/doubled quotes (for example
''{2}\") which will render incorrectly in user-facing messages. Please normalize quoting around placeholders (for example, use\"{2}\"consistently) and ensure opening/closing quotes match in each string.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets include mismatched/doubled quotes (for example
''{2}\") which will render incorrectly in user-facing messages. Please normalize quoting around placeholders (for example, use\"{2}\"consistently) and ensure opening/closing quotes match in each string.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets include mismatched/doubled quotes (for example
''{2}\") which will render incorrectly in user-facing messages. Please normalize quoting around placeholders (for example, use\"{2}\"consistently) and ensure opening/closing quotes match in each string.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish targets include mismatched/doubled quotes (for example
''{2}\") which will render incorrectly in user-facing messages. Please normalize quoting around placeholders (for example, use\"{2}\"consistently) and ensure opening/closing quotes match in each string.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - There is a typo in the placeholder/quoting segment (
{2}v) and the quotes appear unbalanced. This will produce a malformed message at runtime; please correct it to reference{2}and close the quotes properly.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf:1 - Missing space between
{1}anddeklarovaného, which will concatenate words in the rendered message. Insert a space after the{1}placeholder.
src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.zh-Hans.xlf:1 - This change replaces an existing localized target with an English
state=\"new\"target, which is a functional localization regression (users of this locale will now see English). If the prior translation was outdated, consider updating/redoing the translation and keepingstate=\"needs-review-translation\"(ortranslatedonce validated) rather than reverting to English. This appears to affect multipleExtensionResources.*.xlffiles for the same trans-unit.
src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.zh-Hant.xlf:262 - This reverts a previously localized string back to the English source with
state=\"new\". If the old translation was incorrect (it previously stated the default as 'on' while the source says 'off'), it would be better to correct the translation to match the source rather than dropping localization entirely. This same regression appears across multipleAzureDevOpsResources.*.xlffiles forGroupsOptionDescription.
<target state="new">Enable or disable per-assembly Azure DevOps log groups ('on' or 'off'). Defaults to 'off'. Requires '--report-azdo'.</target>
Amaury Levé (Evangelink)
approved these changes
Aug 11, 2026
Amaury Levé (Evangelink)
enabled auto-merge (squash)
August 11, 2026 13:03
dotnet-oneloc-localization
Bot
changed the title
dotnet-oneloc-localization
Bot
changed the title
dotnet-oneloc-localization
Bot
changed the title
…: Build ID 3045042
…: Build ID 3045042
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 65 out of 65 changed files in this pull request and generated no new comments.
Suppressed comments (12)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- These German translations appear to reorder the placeholders compared to the
<note>(e.g.,{0}should be assembly path,{1}extension name,{2}manifest path). As written, runtime messages will show incorrect values. Please update the text so placeholders map 1:1 to the source/note ordering (keep{0}as assembly path,{1}as extension display name,{2}as manifest file path).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - These German translations appear to reorder the placeholders compared to the
<note>(e.g.,{0}should be assembly path,{1}extension name,{2}manifest path). As written, runtime messages will show incorrect values. Please update the text so placeholders map 1:1 to the source/note ordering (keep{0}as assembly path,{1}as extension display name,{2}as manifest file path).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - These German translations appear to reorder the placeholders compared to the
<note>(e.g.,{0}should be assembly path,{1}extension name,{2}manifest path). As written, runtime messages will show incorrect values. Please update the text so placeholders map 1:1 to the source/note ordering (keep{0}as assembly path,{1}as extension display name,{2}as manifest file path).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish
<target>strings have mismatched quoting (e.g.,''{2}\",''{0}\",''{1}\") which will render incorrectly in user-facing output and may confuse users. Please normalize quotes consistently (e.g.,{n}wrapped with the same quote style used elsewhere) and remove the stray doubled apostrophes.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish
<target>strings have mismatched quoting (e.g.,''{2}\",''{0}\",''{1}\") which will render incorrectly in user-facing output and may confuse users. Please normalize quotes consistently (e.g.,{n}wrapped with the same quote style used elsewhere) and remove the stray doubled apostrophes.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish
<target>strings have mismatched quoting (e.g.,''{2}\",''{0}\",''{1}\") which will render incorrectly in user-facing output and may confuse users. Please normalize quotes consistently (e.g.,{n}wrapped with the same quote style used elsewhere) and remove the stray doubled apostrophes.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish
<target>strings have mismatched quoting (e.g.,''{2}\",''{0}\",''{1}\") which will render incorrectly in user-facing output and may confuse users. Please normalize quotes consistently (e.g.,{n}wrapped with the same quote style used elsewhere) and remove the stray doubled apostrophes.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - Several Spanish
<target>strings have mismatched quoting (e.g.,''{2}\",''{0}\",''{1}\") which will render incorrectly in user-facing output and may confuse users. Please normalize quotes consistently (e.g.,{n}wrapped with the same quote style used elsewhere) and remove the stray doubled apostrophes.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - There is a typo in the Spanish translation:
\"{2}vhas an extravand the closing quote is missing/shifted, which will display broken text at runtime. Please correct it to keep{2}intact and properly quoted.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf:1 - Missing space in Czech translation between
{1}anddeklarovaného, which will render concatenated text. Please insert a space so the sentence reads correctly.
src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.ja.xlf:1 - This change replaces an existing localized Japanese description (
state=\"needs-review-translation\") with an Englishstate=\"new\"target, which is a localization regression for users. If the previous translation is outdated, consider keeping it and updating only the changed clauses/URL (or keep it marked as needing review) instead of reverting entirely to English. The same pattern appears across multiple PackagedApp locale files in this PR.
src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.ja.xlf:263 - This replaces a previously localized
GroupsOptionDescriptionwith an Englishstate=\"new\"target, which is a localization regression. If the previous translation was incorrect (some locales indicated the wrong default), it’s better to correct the translation (including the 'Defaults to off' part) rather than resetting to English. The same change occurs across several AzureDevOpsResources locale files in this PR.
<target state="new">Enable or disable per-assembly Azure DevOps log groups ('on' or 'off'). Defaults to 'off'. Requires '--report-azdo'.</target>
<note>{Locked="on"}{Locked="off"}{Locked="--report-azdo"}</note>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 65 out of 65 changed files in this pull request and generated no new comments.
Suppressed comments (8)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- The German translations swap placeholder meanings: per the note, {0}=assembly path, {1}=extension name, {2}=manifest path, but the text reads as if {0}=manifest and {2}=extension. This will report incorrect information to users. Please update the translations so placeholders match the source semantics (e.g., manifest refers to {2}, extension to {1}, assembly to {0}).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - This translation also swaps placeholders: the note says {0}=extension display name and {1}=manifest path, but the German sentence uses {0} as the manifest and {1} as the extension. Please swap the placeholders in the translation text to align with the source.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf:1 - The Spanish translations have mismatched quoting around the {2} placeholder ("''{2}"), which will render incorrectly. Please fix the quotes to be consistent (e.g., use "{2}" or '{2}' as used elsewhere).
src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.zh-Hant.xlf:1 - This resets a previously localized description to English (
state=\"new\"), reducing localization coverage for the PackagedApp extension. If the source text changed and needs re-translation, consider keeping the existing translation and marking it asneeds-review-translation(or updating it) rather than dropping to English.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf:1 - There is a missing space between
{1}and the following word ({1}deklarovaného). Please insert a space so the rendered message is readable.
src/Platform/Microsoft.Testing.Extensions.Retry/Resources/xlf/ExtensionResources.tr.xlf:1 - New resource entries are added but left untranslated (
state=\"new\"with English target) in a locale-specific file. If these strings are user-facing, please provide translations (or keep prior localized equivalents) so the Retry extension remains localized for Turkish.
src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/Resources/xlf/AzureDevOpsResources.zh-Hant.xlf:264 - This change replaces an existing localized target with the English source and marks it as
state=\"new\", which regresses localization for zh-Hant. If the prior translation was reverted due to the incorrect default value ('on' vs 'off'), consider updating the translation to reflect the correct default ('off') rather than resetting to English.
<trans-unit id="GroupsOptionDescription">
<source>Enable or disable per-assembly Azure DevOps log groups ('on' or 'off'). Defaults to 'off'. Requires '--report-azdo'.</source>
<target state="new">Enable or disable per-assembly Azure DevOps log groups ('on' or 'off'). Defaults to 'off'. Requires '--report-azdo'.</target>
<note>{Locked="on"}{Locked="off"}{Locked="--report-azdo"}</note>
</trans-unit>
src/Analyzers/MSTest.Analyzers/xlf/Resources.es.xlf:113
- Grammar agreement issue in Spanish: "la condición infringido" should agree in gender (e.g., "la condición infringida").
<trans-unit id="AssertionArgsShouldAvoidConditionalAccessDescription">
<source>Conditional access adds an implicit null branch to an assertion, which can hide whether the object was unexpectedly null. A separate null assertion makes failures identify the violated condition.</source>
<target state="translated">El acceso condicional agrega una rama null implícita a una aserción, lo que puede ocultar si el objeto era null de forma inesperada. Una aserción null independiente hace que los errores identifiquen la condición infringido.</target>
<note>{Locked="null"}</note>
…: Build ID 3050674
dotnet-oneloc-localization
Bot
changed the title
…: Build ID 3050772
…: Build ID 3050772
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 128 out of 128 changed files in this pull request and generated no new comments.
Suppressed comments (6)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- The German translations reorder placeholders compared to the source, which will substitute the wrong values at runtime (e.g., manifest path displayed where assembly path should be). Update the target strings so placeholder indices match the meaning described in the
<note>(AssemblyLoadFailed/NotFound: {0}=assembly path, {1}=extension display name, {2}=manifest file path; CannotRegisterTestFramework: {0}=extension display name, {1}=manifest file path).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - This new German resource entry is still English with
state=\"new\", while surrounding entries are already translated. If this file is used in shipping builds, users will see English for this error. Provide a German translation (keeping the locked property names intact) and set the target state appropriately.
src/Platform/Microsoft.Testing.Extensions.Retry/Resources/xlf/ExtensionResources.zh-Hans.xlf:1 - These new zh-Hans targets are still English (
state=\"new\"). If the Retry extension emits these messages to end users, this introduces mixed-language output. Consider adding Simplified Chinese translations (or marking them as needing review per the repo’s localization workflow) before release.
src/Analyzers/MSTest.Analyzers/xlf/Resources.fr.xlf:425 - These newly added analyzer descriptions in the French resource file are still English (
state=\"new\"). Since analyzer titles/messages are user-facing in IDE/build output, this results in partially localized diagnostics. Add French translations (preserving the locked tokens) and update the target state.
<trans-unit id="InheritedMemberFromDifferentMSTestVersionDescription">
<source>MSTest resolves the inherited lifecycle attributes '[TestInitialize]', '[TestCleanup]', '[ClassInitialize]' and '[ClassCleanup]', and the test attributes '[TestMethod]' and '[DataTestMethod]', by their exact framework type identity. The framework assembly was renamed from 'Microsoft.VisualStudio.TestPlatform.TestFramework' to 'MSTest.TestFramework', so a base class compiled against a different MSTest major exposes these attributes with a different type identity. When such a base is inherited by a test class compiled against another version, the inherited setup and cleanup do not run and inherited test methods are not discovered, with no build error and no discovery error. Recompile the base assembly against the same MSTest version as the test project.</source>
<target state="new">MSTest resolves the inherited lifecycle attributes '[TestInitialize]', '[TestCleanup]', '[ClassInitialize]' and '[ClassCleanup]', and the test attributes '[TestMethod]' and '[DataTestMethod]', by their exact framework type identity. The framework assembly was renamed from 'Microsoft.VisualStudio.TestPlatform.TestFramework' to 'MSTest.TestFramework', so a base class compiled against a different MSTest major exposes these attributes with a different type identity. When such a base is inherited by a test class compiled against another version, the inherited setup and cleanup do not run and inherited test methods are not discovered, with no build error and no discovery error. Recompile the base assembly against the same MSTest version as the test project.</target>
src/Analyzers/MSTest.Analyzers/xlf/Resources.fr.xlf:1382
- These newly added analyzer descriptions in the French resource file are still English (
state=\"new\"). Since analyzer titles/messages are user-facing in IDE/build output, this results in partially localized diagnostics. Add French translations (preserving the locked tokens) and update the target state.
<trans-unit id="UseExecutableConditionAttributeInsteadOfProcessCheckDescription">
<source>Test methods that use 'File.Exists' to skip a test before starting the same executable with 'Process.Start' should use the '[ExecutableCondition]' attribute instead. This attribute provides a declarative and discoverable tool requirement and reports an early return as a skipped test rather than a passed test.</source>
<target state="new">Test methods that use 'File.Exists' to skip a test before starting the same executable with 'Process.Start' should use the '[ExecutableCondition]' attribute instead. This attribute provides a declarative and discoverable tool requirement and reports an early return as a skipped test rather than a passed test.</target>
src/Analyzers/MSTest.Analyzers.CodeFixes/xlf/CodeFixResources.tr.xlf:217
- This new Turkish code-fix label is still English with
state=\"new\". Code fix titles are directly visible in IDE UI, so this will appear mixed-language. Provide a Turkish translation while keeping the locked token unchanged.
<trans-unit id="UseExecutableConditionAttributeInsteadOfProcessCheckFix">
<source>Use '[ExecutableCondition]' attribute</source>
<target state="new">Use '[ExecutableCondition]' attribute</target>
<note>{Locked="'[ExecutableCondition]'"}</note>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 128 out of 128 changed files in this pull request and generated no new comments.
Suppressed comments (6)
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1
- The
{0}/{1}/{2}placeholders in these German translations appear to be reordered compared to the corresponding<note>expectations (e.g.,{0}should be the resolved assembly path,{1}the extension display name,{2}the manifest path). This will produce incorrect runtime messages. Align the translation text so the placeholder indices match the source meaning (keep indices consistent; adjust wording rather than swapping placeholders).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - The
{0}/{1}/{2}placeholders in these German translations appear to be reordered compared to the corresponding<note>expectations (e.g.,{0}should be the resolved assembly path,{1}the extension display name,{2}the manifest path). This will produce incorrect runtime messages. Align the translation text so the placeholder indices match the source meaning (keep indices consistent; adjust wording rather than swapping placeholders).
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf:1 - The
{0}/{1}/{2}placeholders in these German translations appear to be reordered compared to the corresponding<note>expectations (e.g.,{0}should be the resolved assembly path,{1}the extension display name,{2}the manifest path). This will produce incorrect runtime messages. Align the translation text so the placeholder indices match the source meaning (keep indices consistent; adjust wording rather than swapping placeholders).
src/Platform/Microsoft.Testing.Extensions.Retry/Resources/xlf/ExtensionResources.de.xlf:1 - These newly added retry strings are left in English with
state=\"new\"in a localized (de) resource file. If shipped as-is, German users will see English UI/console output for these messages. Provide German translations and mark themstate=\"translated\"(or follow the repo’s standard workflow for incomplete translations, e.g.,needs-review-translation).
src/Analyzers/MSTest.Analyzers.CodeFixes/xlf/CodeFixResources.zh-Hant.xlf:217 - This new code-fix label is still English (
state=\"new\") in the zh-Hant resource file. Translate the<target>to zh-Hant (keeping the locked token unchanged) and set the state totranslated(or the repo’s expected review state) to avoid mixed-language UX.
<trans-unit id="UseExecutableConditionAttributeInsteadOfProcessCheckFix">
<source>Use '[ExecutableCondition]' attribute</source>
<target state="new">Use '[ExecutableCondition]' attribute</target>
<note>{Locked="'[ExecutableCondition]'"}</note>
</trans-unit>
src/Analyzers/MSTest.Analyzers/xlf/Resources.fr.xlf:425
- A new analyzer description was added to the French resource file but remains English with
state=\"new\". If analyzer documentation/tooling surfaces these strings in French environments, this will be inconsistent. Add a French translation (preserving locked tokens) and mark ittranslated(or the repo’s expected review state).
<trans-unit id="InheritedMemberFromDifferentMSTestVersionDescription">
<source>MSTest resolves the inherited lifecycle attributes '[TestInitialize]', '[TestCleanup]', '[ClassInitialize]' and '[ClassCleanup]', and the test attributes '[TestMethod]' and '[DataTestMethod]', by their exact framework type identity. The framework assembly was renamed from 'Microsoft.VisualStudio.TestPlatform.TestFramework' to 'MSTest.TestFramework', so a base class compiled against a different MSTest major exposes these attributes with a different type identity. When such a base is inherited by a test class compiled against another version, the inherited setup and cleanup do not run and inherited test methods are not discovered, with no build error and no discovery error. Recompile the base assembly against the same MSTest version as the test project.</source>
<target state="new">MSTest resolves the inherited lifecycle attributes '[TestInitialize]', '[TestCleanup]', '[ClassInitialize]' and '[ClassCleanup]', and the test attributes '[TestMethod]' and '[DataTestMethod]', by their exact framework type identity. The framework assembly was renamed from 'Microsoft.VisualStudio.TestPlatform.TestFramework' to 'MSTest.TestFramework', so a base class compiled against a different MSTest major exposes these attributes with a different type identity. When such a base is inherited by a test class compiled against another version, the inherited setup and cleanup do not run and inherited test methods are not discovered, with no build error and no discovery error. Recompile the base assembly against the same MSTest version as the test project.</target>
<note>{Locked="MSTest"}{Locked="'[TestInitialize]'"}{Locked="'[TestCleanup]'"}{Locked="'[ClassInitialize]'"}{Locked="'[ClassCleanup]'"}{Locked="'[TestMethod]'"}{Locked="'[DataTestMethod]'"}{Locked="'Microsoft.VisualStudio.TestPlatform.TestFramework'"}{Locked="'MSTest.TestFramework'"}</note>
</trans-unit>
…: Build ID 3051589
Contributor
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 128 out of 128 changed files in this pull request and generated no new comments.
Suppressed comments (2)
src/Platform/Microsoft.Testing.Extensions.CtrfReport/Resources/xlf/ExtensionResources.pl.xlf:17
- The Polish phrase uses the nominative plural
plikiafterliczbę; this construction requires the genitive pluralplików. As written, the localized artifact description is grammatically incorrect.
src/Platform/Microsoft.Testing.Extensions.PackagedApp/Resources/xlf/ExtensionResources.cs.xlf:77 - The Czech adjective does not agree with the feminine singular noun
aplikace:nezabalené aplikaceshould benezabalená aplikace. The current package description is grammatically incorrect.
dotnet-oneloc-localization
Bot
changed the title
…: Build ID 3052544
…: Build ID 3052544
…: Build ID 3052544
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 128 out of 128 changed files in this pull request and generated no new comments.
Suppressed comments (3)
src/Platform/Microsoft.Testing.Extensions.JUnitReport/Resources/xlf/ExtensionResources.pl.xlf:22
- The Polish target is grammatically incorrect: the count must govern the genitive plural
plików, and the placeholder should be part of that phrase. Please have the localization pipeline produce the equivalent of “Scalono {0} plików raportu XML JUnit”.
src/Platform/Microsoft.Testing.Extensions.HtmlReport/Resources/xlf/ExtensionResources.pl.xlf:17 - The Polish target is grammatically incorrect: the count must govern the genitive plural
plików, and the placeholder should be part of that phrase. Please have the localization pipeline produce the equivalent of “Scalono {0} plików raportu HTML”.
src/Platform/Microsoft.Testing.Extensions.CtrfReport/Resources/xlf/ExtensionResources.pl.xlf:17 - The Polish phrase
liczbę plikiis ungrammatical;liczbęrequires the genitive pluralplików. Please correct this through the localization pipeline, for example to “Liczba scalonych plików CTRF: {0}”.
Contributor
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
…: Build ID 3052544
…: Build ID 3052544
…: Build ID 3052544
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 129 out of 129 changed files in this pull request and generated no new comments.
Suppressed comments (1)
src/Platform/Microsoft.Testing.Extensions.CtrfReport/Resources/xlf/ExtensionResources.pl.xlf:17
- The noun has the wrong case after
liczbę;plikishould be the genitiveplików, otherwise the translated status message is grammatically incorrect.
Contributor
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is the pull request automatically created by the OneLocBuild task in the build process to check-in localized files generated based upon translation source files (.lcl files) handed-back from the downstream localization pipeline. If there are issues in translations, visit https://aka.ms/icxLocBug and log bugs for fixes. The OneLocBuild wiki is https://aka.ms/onelocbuild and the localization process in general is documented at https://aka.ms/AllAboutLoc.