Skip to content

Conversation

@srutzky
Copy link
Contributor

@srutzky srutzky commented Aug 24, 2019

The term "the Unix Standard" is incorrect and should instead be "the Unicode Standard". I fixed 4 instances of this mistake, all in "GetUnicodeCategory". This mistake appears to have been introduced in the following commit on March 21, 2017:

cd8c569#diff-206ae69d9a4bfeaf7052220b8e8cde2a

The term "the Unix Standard" is incorrect and should instead be "the Unicode Standard". I fixed 4 instances of this mistake, all in "GetUnicodeCategory". This mistake appears to have been introduced in the following commit on March 21, 2017:

dotnet@cd8c569#diff-206ae69d9a4bfeaf7052220b8e8cde2a
@rpetrusha rpetrusha added this to the August 2019 milestone Aug 26, 2019
@rpetrusha rpetrusha added the ✨ 1st-time dotnet-api-docs contributor! Indicates PRs from new contributors to the dotnet-api-docs repository label Aug 26, 2019
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

Thank you, @srutzky, for contributing to the dotnet/docs repo and correcting the misnamed references to the Unicode Standard. We'll merge your PR now. Your changes should be live on docs.microsoft.com in the next day or two.

@rpetrusha rpetrusha merged commit a464a88 into dotnet:master Aug 26, 2019
@srutzky srutzky deleted the patch-1 branch September 6, 2019 12:11
@srutzky
Copy link
Contributor Author

srutzky commented Sep 6, 2019

@rpetrusha You're welcome 😺

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ 1st-time dotnet-api-docs contributor! Indicates PRs from new contributors to the dotnet-api-docs repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants