Skip to content

Conversation

CarnaViire
Copy link
Member

Add missing <inheritdoc/> and <returns> for System.Net.* APIs

Fixes dotnet/runtime#120497
Fixes dotnet/runtime#120498
Fixes dotnet/runtime#120499

@CarnaViire CarnaViire requested a review from a team as a code owner October 9, 2025 19:11
@Copilot Copilot AI review requested due to automatic review settings October 9, 2025 19:11
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates XML documentation for System.Net.* APIs to improve documentation completeness for .NET 10. The changes focus on adding missing <inheritdoc/> tags and providing proper <returns> descriptions for API methods.

  • Added <inheritdoc/> tags to methods that inherit documentation from base classes or interfaces
  • Replaced placeholder "To be added." return descriptions with specific documentation for IPAddress validation methods
  • Updated documentation across networking-related classes including IPNetwork, IPAddress, WebSocketStream, and SslStream

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
xml/System.Net/IPNetwork.xml Added <inheritdoc/> tag to ToString method documentation
xml/System.Net/IPAddress.xml Added proper <returns> descriptions for IsValid methods and <inheritdoc/> tag for ToString method
xml/System.Net.WebSockets/WebSocketStream.xml Added <inheritdoc/> tag to Dispose method documentation
xml/System.Net.Security/SslStream.xml Added <inheritdoc/> tags to Read, Write, and WriteTimeout property documentation

Copy link
Contributor

Tagging subscribers to this area: @dotnet/ncl

@CarnaViire CarnaViire requested a review from gewarren October 10, 2025 12:27
Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

LGTM

@CarnaViire CarnaViire merged commit 9b4828c into dotnet:main Oct 10, 2025
6 checks passed
@CarnaViire CarnaViire deleted the system-net-doc-net10-apis branch October 10, 2025 23:20
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.

System.Net docs for .NET 10.0 APIs System.Net.WebSockets docs for .NET 10.0 APIs System.Net.Security docs for .NET 10.0 APIs

2 participants