Skip to content
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

[Localization] Using old translations for our MSBuild localization #11478

Merged
merged 2 commits into from
May 8, 2021

Conversation

tj-devel709
Copy link
Member

@tj-devel709 tj-devel709 commented May 7, 2021

This PR takes the old translations we used in MSBuild and adds them to our new localization flow.
This will allow us to not drop translations for any customers until we get the localization strings back from the Loc team in the new OneLocBuild localization pipeline.

This also re-enables 2 of the MSBuild localization tests that I ignored in a previous PR.
I still cannot remove the ignores for any mtouch localization tests since we did not have those translations.
There are also 4 new languages added from OneLocBuild that do not have existing translations so I needed to keep one of the msbuild tests ignored.

@tj-devel709 tj-devel709 added the not-notes-worthy Ignore for release notes label May 7, 2021
@tj-devel709 tj-devel709 added this to the Future milestone May 7, 2021
@dalexsoto dalexsoto requested a review from mandel-macaque May 7, 2021 16:36
Copy link
Member

@dalexsoto dalexsoto left a comment

Choose a reason for hiding this comment

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

Nice

@mandel-macaque
Copy link
Member

Now we are talking :)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

✅ [PR Build] Tests passed on Build. ✅

Tests passed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

🎉 All 107 tests passed 🎉

Pipeline on Agent XAMBOT-1094.BigSur'
Merge de63dd6 into 6cf9d78

@tj-devel709 tj-devel709 merged commit 4aa8d9a into dotnet:main May 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-notes-worthy Ignore for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants