Skip to content

Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 2972073#8107

Merged
Evangelink merged 3 commits into
mainfrom
locfiles/b24e8953-c9b4-4db2-aadd-4a895539da84
May 11, 2026
Merged

Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 2972073#8107
Evangelink merged 3 commits into
mainfrom
locfiles/b24e8953-c9b4-4db2-aadd-4a895539da84

Conversation

@dotnet-bot
Copy link
Copy Markdown
Collaborator

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.

Copilot AI review requested due to automatic review settings May 11, 2026 12:53
@github-actions github-actions Bot enabled auto-merge (squash) May 11, 2026 12:53
Copy link
Copy Markdown
Contributor

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 is an automated OneLocBuild check-in updating localized .xlf files for Microsoft.Testing.Platform and MSTest analyzer code-fix resources.

Changes:

  • Updates multiple PlatformResources.*.xlf files for CompositeServiceFactoryInstanceAlreadyRegistered, including marking the translation as translated.
  • Updates Italian CodeFixResources.it.xlf for the RemoveDuplicateDataRowFix localized string.

Reviewed changes

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

Show a summary per file
File Description
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.zh-Hant.xlf Updates zh-Hant localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.zh-Hans.xlf Updates zh-Hans localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.tr.xlf Updates tr localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.ru.xlf Updates ru localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.pt-BR.xlf Updates pt-BR localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.pl.xlf Updates pl localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.ko.xlf Updates ko localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.ja.xlf Updates ja localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.it.xlf Updates it localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.fr.xlf Updates fr localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.es.xlf Updates es localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.de.xlf Updates de localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Platform/Microsoft.Testing.Platform/Resources/xlf/PlatformResources.cs.xlf Updates cs localized entry for CompositeServiceFactoryInstanceAlreadyRegistered.
src/Analyzers/MSTest.Analyzers.CodeFixes/xlf/CodeFixResources.it.xlf Updates Italian localized title for RemoveDuplicateDataRowFix.

Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">已註冊 'CompositeExtensonFactory' 的同一執行個體</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">“CompositeExtensonFactory”的同一实例已注册</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">Aynı 'CompositeExtensonFactory' örneği zaten kayıtlı</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">Такой же экземпляр CompositeExtensonFactory уже зарегистрирован</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">A mesma instância de “CompositeExtensonFactory” já está registrada</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">La même instance de « CompositeExtensonFactory » est déjà inscrite</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">La misma instancia de "CompositeExtensonFactory" ya está registrada</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">Die gleiche Instanz von "CompositeExtensonFactory" ist bereits registriert</target>
Comment on lines +166 to +167
<source>The same instance of 'CompositeExtensonFactory' is already registered</source>
<target state="translated">Už je zaregistrovaná stejná instance CompositeExtensonFactory.</target>
<trans-unit id="RemoveDuplicateDataRowFix">
<source>Remove duplicate 'DataRow'</source>
<target state="translated">Rimuovere 'DataRow' duplicato</target>
<target state="translated">Rimuovere ''DataRow'' duplicato</target>
Copy link
Copy Markdown
Member

@Evangelink Evangelink left a comment

Choose a reason for hiding this comment

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

Summary

Workflow: PR Nitpick Reviewer 🔍
Date: 2026-05-11
Repository: microsoft/testfx

Key Findings

This is an automated OneLocBuild localization check-in touching 14 XLF files. Two subtle issues are worth a look:

  1. Important – XLF source/resx mismatch (typo): All 12 PlatformResources.*.xlf files now contain <source>The same instance of 'CompositeExtensonFactory' is already registered</source>CompositeExtensonFactory is missing the letter i. The actual PlatformResources.resx source string spells it correctly as CompositeExtensionFactory. The XLF <source> should mirror the .resx source verbatim to keep the pipeline stable.

  2. Minor – Inconsistent apostrophe escaping in Italian CodeFix translation: CodeFixResources.it.xlf changes 'DataRow'''DataRow'' (doubled apostrophes). This is the correct escaping for String.Format usage, but the English .resx source still uses single apostrophes. If the string is format-interpolated, the source should be updated too; if not, the translation should revert to match.

Recommendations

  • Verify whether the .resx source for CompositeServiceFactoryInstanceAlreadyRegistered was intentionally changed upstream or if a typo was silently introduced in the localization roundtrip.
  • Clarify String.Format usage for RemoveDuplicateDataRowFix and align apostrophe escaping in both source and all translations.

Generated by PR Nitpick Reviewer

🔍 Meticulously inspected by PR Nitpick Reviewer 🔍

@@ -163,8 +163,8 @@
<note />
</trans-unit>
<trans-unit id="CompositeServiceFactoryInstanceAlreadyRegistered">
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nitpick (Important): The <source> element now reads 'CompositeExtensonFactory' (missing the letter i), but the canonical English string in PlatformResources.resx is 'CompositeExtensionFactory' (correct spelling). XLIFF <source> elements should stay in sync with the .resx source to keep the localization pipeline consistent.

The same discrepancy is present across all 12 language XLF files updated in this PR (cs, de, es, fr, it, ja, ko, pl, pt-BR, ru, tr, zh-Hans, zh-Hant).

Why it matters: If the XLF <source> diverges from the .resx source, the next OneLocBuild run may flag all these strings as "changed" and re-queue them for translation unnecessarily, or it may silently mask a misspelling that could surface in diagnostic messages.

@@ -69,7 +69,7 @@
</trans-unit>
<trans-unit id="RemoveDuplicateDataRowFix">
<source>Remove duplicate 'DataRow'</source>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nitpick (Minor): The Italian translation now uses ''DataRow'' (doubled apostrophes) while the English source string in the .resx is Remove duplicate 'DataRow' (single apostrophes).

In .NET resource strings, apostrophes must be doubled ('') when the value is consumed through String.Format/composite formatting; they should remain single otherwise. If this string is used with String.Format, the source .resx entry should also use ''DataRow'' for consistency. If it is not, the Italian translation should revert to single quotes to match.

Why it matters: Mismatched escaping leads to strings rendering differently in different locales — Italian users would see DataRow with no surrounding quotes if String.Format is applied.

@dotnet-bot dotnet-bot changed the title Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 2972050 Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 2972063 May 11, 2026
@dotnet-bot dotnet-bot changed the title Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 2972063 Localized file check-in by OneLocBuild Task: Build definition ID 1218: Build ID 2972073 May 11, 2026
@Evangelink Evangelink disabled auto-merge May 11, 2026 13:20
@Evangelink Evangelink merged commit 13d6e17 into main May 11, 2026
22 of 31 checks passed
@Evangelink Evangelink deleted the locfiles/b24e8953-c9b4-4db2-aadd-4a895539da84 branch May 11, 2026 13:20
Evangelink pushed a commit that referenced this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants