Skip to content

Conversation

@ManickaP
Copy link
Member

Fixed some typos and improved wording based on @stephentoub comments in #8472

@ManickaP ManickaP requested review from a team as code owners October 11, 2022 13:46
@ghost ghost assigned ManickaP Oct 11, 2022
@ghost ghost added the area-System.Net label Oct 11, 2022
@ghost
Copy link

ghost commented Oct 11, 2022

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixed some typos and improved wording based on @stephentoub comments in #8472

Author: ManickaP
Assignees: ManickaP
Labels:

area-System.Net

Milestone: -

@opbld30
Copy link

opbld30 commented Oct 11, 2022

Learn Build status updates of commit ad9ac84:

✅ Validation status: passed

File Status Preview URL Details
snippets/cpp/VS_Snippets_Remoting/Httpwebrequest_HaveResponse/CPP/httpwebrequest_haveresponse.cpp ✅Succeeded View
snippets/csharp/System.Net/HttpWebRequest/HaveResponse/httpwebrequest_haveresponse.cs ✅Succeeded View
snippets/visualbasic/VS_Snippets_Remoting/Httpwebrequest_HaveResponse/VB/httpwebrequest_haveresponse.vb ✅Succeeded View
xml/System.Net.Http/HttpClientHandler.xml ✅Succeeded View
xml/System.Net.Http/SocketsHttpHandler.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

@liveans liveans left a comment

Choose a reason for hiding this comment

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

LGTM, few nits

@opbld32
Copy link

opbld32 commented Oct 11, 2022

Learn Build status updates of commit 4d53920:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Net.Http/HttpClientHandler.xml ✅Succeeded View
xml/System.Net.Http/SocketsHttpHandler.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:

@ManickaP ManickaP merged commit 8bad323 into dotnet:main Oct 11, 2022
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.

4 participants