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

Add shared resx files to our shared projects #41775

Merged
merged 1 commit into from
Feb 19, 2020

Conversation

mavasani
Copy link
Contributor

Extracted from #41363
Third follow-up item from #41462

Apart from adding resx files, changes also include:

  1. Moving resource strings duplicated across Workspaces and CodeStyle layer into the shared resx files
  2. Source file changes to use the resource strings from the shared resx.

We should no longer require use of #if CODE_STYLE in the shared layer for the purpose of resource strings.

Extracted from dotnet#41363
Third follow-up item from dotnet#41462

Apart from adding resx files, changes also include:
1. Moving resource strings duplicated across Workspaces and CodeStyle layer into the shared resx files
2. Source file changes to use the resource strings from the shared resx.

We should no longer require use of `#if CODE_STYLE` in the shared layer for the purpose of resource strings.
@mavasani mavasani added this to the 16.6.P1 milestone Feb 19, 2020
@mavasani mavasani requested review from tmat, sharwell, CyrusNajmabadi and a team February 19, 2020 14:55
@mavasani
Copy link
Contributor Author

NOTE: 208 out of the 250 changed files are xlf files. Strongly recommend reviewing with xlf files filtered: Review Link

This is an extremely mechanical change with absolutely no semantic changes.

@CyrusNajmabadi
Copy link
Member

We should no longer require use of #if CODE_STYLE in the shared layer for the purpose of resource strings.

Fantastic!!

This is an extremely mechanical change with absolutely no semantic changes.

This is very appreciated. Thanks for taking the time to do that!

Copy link
Member

@tmat tmat left a comment

Choose a reason for hiding this comment

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

:shipit:

@mavasani mavasani merged commit 15b4cc6 into dotnet:master Feb 19, 2020
@mavasani mavasani deleted the SharedResources branch February 19, 2020 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants