Skip to content

Update description for UploadString() method. #4015

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

Merged
merged 1 commit into from
Mar 23, 2020

Conversation

ericmutta
Copy link
Contributor

The remarks section uses the following text for both the UploadString() and UploadStringAsync() methods:

Sends a String to the resource, without blocking the calling thread.

I think this is a copy-and-paste error because only the UploadStringAsync() method is non-blocking. I have updated the description for UploadString() to follow the pattern used when describing other upload methods.

The remarks section uses the following text for both the `UploadString()` and `UploadStringAsync()` methods:

> Sends a String to the resource, without blocking the calling thread.

I think this is a copy-and-paste error because only the `UploadStringAsync()` method is non-blocking. I have updated the description for `UploadString()` to follow the pattern used when describing other upload methods.
@ericmutta ericmutta requested a review from a team as a code owner March 20, 2020 19:50
@dotnet-bot dotnet-bot added this to the March 2020 milestone Mar 20, 2020
@davidsh davidsh requested a review from gewarren March 20, 2020 20:08
Copy link
Member

@karelz karelz left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @ericmutta

@carlossanlop carlossanlop merged commit 796bf65 into dotnet:master Mar 23, 2020
@ericmutta ericmutta deleted the patch-1 branch March 23, 2020 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants