Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove uses of SR.GetResourceString throughout the libs #87912

Merged
merged 3 commits into from
Jun 23, 2023

Conversation

MichalStrehovsky
Copy link
Member

Fixes #81338.

Cc @dotnet/ilc-contrib @stephentoub

@ghost
Copy link

ghost commented Jun 22, 2023

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
@jkoritzinsky
Copy link
Member

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

@MichalStrehovsky
Copy link
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
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ban SR.GetResourceString
4 participants