Skip to content

Conversation

@aik-jahoda
Copy link
Contributor

@aik-jahoda aik-jahoda marked this pull request as ready for review August 28, 2020 14:27
@aik-jahoda aik-jahoda requested review from a team and BillWagner August 28, 2020 14:27
@opbld32
Copy link

opbld32 commented Aug 28, 2020

Docs Build status updates of commit ce2d2bd:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Net.Http/ByteArrayContent.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld33
Copy link

opbld33 commented Aug 28, 2020

Docs Build status updates of commit ed91056:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Net.Http/ByteArrayContent.xml ✅Succeeded View
xml/System.Net.Http/FormUrlEncodedContent.xml ✅Succeeded View
xml/System.Net.Http/HttpContent.xml ✅Succeeded View
xml/System.Net.Http/MultipartContent.xml ✅Succeeded View
xml/System.Net.Http/MultipartFormDataContent.xml ✅Succeeded View
xml/System.Net.Http/StreamContent.xml ✅Succeeded View
xml/System.Net.Http/StringContent.xml ✅Succeeded View

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Member

@BillWagner BillWagner 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 writing this @aik-jahoda

It LGTM, and I'll :shipit: now.

@BillWagner BillWagner merged commit 752438e into dotnet:master Aug 31, 2020
@aik-jahoda aik-jahoda deleted the jajahoda/SerializeToStreamAsync branch September 2, 2020 12:50
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