Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Remove uses of SR.GetResourceString throughout the libs - #87912

#87912
Merged
MichalStrehovsky merged 3 commits into
dotnet:maindotnet/runtime:mainfrom
MichalStrehovsky:fix81338MichalStrehovsky/runtime:fix81338Copy head branch name to clipboard
Jun 23, 2023
Merged

Remove uses of SR.GetResourceString throughout the libs#87912
MichalStrehovsky merged 3 commits into
dotnet:maindotnet/runtime:mainfrom
MichalStrehovsky:fix81338MichalStrehovsky/runtime:fix81338Copy head branch name to clipboard

Conversation

@MichalStrehovsky

Copy link
Copy Markdown
Member

Fixes #81338.

Cc @dotnet/ilc-contrib @stephentoub

@ghost

ghost commented Jun 22, 2023

Copy link
Copy Markdown

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #81338.

Cc @dotnet/ilc-contrib @stephentoub

Author: MichalStrehovsky
Assignees: -
Labels:

area-NativeAOT-coreclr

Milestone: -

@MichalStrehovsky MichalStrehovsky changed the title Remove uses of SR.GetResourceString through the libs Remove uses of SR.GetResourceString throughout the libs Jun 22, 2023
Comment thread src/libraries/Microsoft.Extensions.Primitives/src/ThrowHelper.cs Outdated
@jkoritzinsky

Copy link
Copy Markdown
Member

Do we want to use the BannedApiAnalyzer to make sure that we don't re-add usages of SR.GetResourceString?

@MichalStrehovsky

Copy link
Copy Markdown
Member Author

Do we want to use the BannedApiAnalyzer to make sure that we don't re-add usages of SR.GetResourceString?

The API is private so the chances are pretty low I think (someone would have to make it public/internal and ignore the comment right above).

@MichalStrehovsky
MichalStrehovsky merged commit a0e670a into dotnet:main Jun 23, 2023
@MichalStrehovsky
MichalStrehovsky deleted the fix81338 branch June 23, 2023 02:51
@ghost ghost locked as resolved and limited conversation to collaborators Jul 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ban SR.GetResourceString

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.