Skip to content

Conversation

@aik-jahoda
Copy link
Contributor

Summary

Add missing HttpContent documentation

part of: dotnet/runtime#41446

@opbld31

This comment has been minimized.

@carlossanlop
Copy link
Contributor

@dotnet/docs do you know why this PR hit all those warnings? Normally the warnings belong to the changes in the PR, not to the whole repo.

Copy link
Contributor

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

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

Changes look good, @aik-jahoda , thanks for submitting the PR. Let's just wait for @dotnet/docs input on why the build hit all those warnings.

@carlossanlop carlossanlop requested review from a team August 31, 2020 23:21
@aik-jahoda
Copy link
Contributor Author

@dotnet/docs do you know why this PR hit all those warnings? Normally the warnings belong to the changes in the PR, not to the whole repo.

@carlossanlop, it also doesn't publish the preview to the review.docs.microsoft.com

@aik-jahoda aik-jahoda closed this Sep 1, 2020
@aik-jahoda
Copy link
Contributor Author

Reopening to trigger build.

@aik-jahoda aik-jahoda reopened this Sep 1, 2020
@opbld30
Copy link

opbld30 commented Sep 1, 2020

Docs Build status updates of commit 6b63a8f:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Net.Http/HttpContent.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:

@carlossanlop carlossanlop merged commit 4a6f04a into dotnet:master Sep 1, 2020
@aik-jahoda aik-jahoda deleted the jajahoda/HttpContentMissingApi 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