Skip to content

Conversation

@liveans
Copy link
Member

@liveans liveans commented Nov 7, 2022

Summary

Updates on snippets and formatting example files.

Fixes #7843

@liveans liveans requested review from a team as code owners November 7, 2022 10:28
@ghost ghost assigned liveans Nov 7, 2022
@ghost
Copy link

ghost commented Nov 7, 2022

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

Issue Details

Summary

Updates on snippets and formatting example files.

Fixes #7843

Author: liveans
Assignees: liveans
Labels:

area-System.Net.Sockets

Milestone: -

@opbld31

This comment was marked as outdated.

@liveans
Copy link
Member Author

liveans commented Nov 7, 2022

You can ignore snippet build fail, because it says Project file is missing for C++.

@opbld30
Copy link

opbld30 commented Nov 7, 2022

Learn Build status updates of commit a618f8b:

✅ Validation status: passed

File Status Preview URL Details
snippets/cpp/VS_Snippets_Remoting/Socket_Socket_Options/CPP/source.cpp ✅Succeeded View
snippets/csharp/System.Net.Sockets/LingerOption/LingerTime/source.cs ✅Succeeded View
snippets/visualbasic/VS_Snippets_Remoting/Socket_Socket_Options/VB/source.vb ✅Succeeded View
xml/System.Net.Sockets/Socket.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
Contributor

@antonfirsov antonfirsov left a comment

Choose a reason for hiding this comment

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

LGTM

@liveans liveans merged commit 2a6ecce into dotnet:main Nov 7, 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.

Socket.SetSocketOption docs are misleading/confusing

4 participants