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

Update CannotCreateNormalize resource name in resource files #1134

Merged
merged 1 commit into from
Jun 23, 2021

Conversation

Wraith2
Copy link
Contributor

@Wraith2 Wraith2 commented Jun 23, 2021

In #1041 I noticed a CI error https://github.com/dotnet/SqlClient/pull/1041/checks?check_run_id=2889045989 which indicates that a string resource can't be found. This is an update from #1057 which changed the name from Sql_CanotCreateNormalizer to SQL_CannotCreateNormalizer but these seem to have been reverted at some point since. This PR just makes sure the incorrect version no longer exists in the project.

Can we get this reviewed and merged asap to prevent CI problems affecting other PRs? @cheenamalhotra

@cheenamalhotra cheenamalhotra added this to the 4.0.0-preview1 milestone Jun 23, 2021
Copy link
Member

@cheenamalhotra cheenamalhotra left a comment

Choose a reason for hiding this comment

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

FYI for future changes, all localized versions of Strings.resx need not be changed. We have daily CI builds to sync those files.
Only english version (Strings.resx) should be updated.

@cheenamalhotra cheenamalhotra merged commit 95d23bb into dotnet:main Jun 23, 2021
@Wraith2 Wraith2 deleted the fix-sqlnormalizerresource branch June 23, 2021 19:40
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.

4 participants