Skip to content

[AsmDiff]Added Localized Content for markdown header and Language Option#7173

Merged
ericstj merged 2 commits intodotnet:mainfrom
Herrmel:AddResourcesForLocalizedContent
Apr 6, 2021
Merged

[AsmDiff]Added Localized Content for markdown header and Language Option#7173
ericstj merged 2 commits intodotnet:mainfrom
Herrmel:AddResourcesForLocalizedContent

Conversation

@Herrmel
Copy link
Contributor

@Herrmel Herrmel commented Apr 1, 2021

Hello its me from #7164
Adding <StrongNameKeyId>Open</StrongNameKeyId> did fix the problems with signing, thank you.

I tried setting the language for the generated comments in the ResourceDesigner, but could not find a way without installing and changing the language of VisualStudio. Maybe you could run the generation once and commit the new output if you want it in english.

@Anipik @safern @ericstj

@dnfadmin
Copy link

dnfadmin commented Apr 1, 2021

CLA assistant check
All CLA requirements met.

@ericstj ericstj requested a review from Anipik April 1, 2021 15:41
@@ -0,0 +1,91 @@
//------------------------------------------------------------------------------
Copy link
Member

Choose a reason for hiding this comment

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

We don't typically check these in, we have a generator that runs during the build to produce them. You can omit this file and set GenerateSource: https://github.com/dotnet/arcade/blob/6cc4c1e9e23d5e65e88a8a57216b3d91e9b3d8db/Documentation/ArcadeSdk.md#generateresxsource-bool

@ericstj
Copy link
Member

ericstj commented Apr 1, 2021

Very cool, thank you for the contribution! I made a couple notes on how to proceed. I also went ahead and implemented the suggestions if you'd like to take them: ericstj@fd5f861

@Herrmel
Copy link
Contributor Author

Herrmel commented Apr 6, 2021

I added your changes to this PR, actually I've never even heard of Xliff before. Thank you.

@ericstj
Copy link
Member

ericstj commented Apr 6, 2021

I've never even heard of Xliff before

This happens to be the infrastructure we use throughout dotnet repos for localization. You can learn more about it here: https://github.com/dotnet/xliff-tasks

@ericstj ericstj merged commit 680ab44 into dotnet:main Apr 6, 2021
@Herrmel Herrmel deleted the AddResourcesForLocalizedContent branch April 7, 2021 08:40
akoeplinger pushed a commit to akoeplinger/arcade that referenced this pull request Apr 12, 2021
…ion (dotnet#7173)

* Added Localized Content for markdown header and Language Option

* Applied changes from ericstj@fd5f861
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