Skip to content

Conversation

@gewarren
Copy link
Contributor

@gewarren gewarren commented Jan 9, 2026

Contributes to dotnet/docs#49545.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-io

Copy link
Contributor

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 the MemoryStream documentation to reflect .NET 11 changes regarding length and capacity exceptions. The primary focus is documenting the new maximum value (0x7FFFFFC7 bytes) for capacity and length operations introduced in .NET 11.

Key changes include:

  • Adding .NET 11-specific exception documentation for Capacity property and SetLength method
  • Fixing xref formatting by removing .%2A suffixes from property references
  • Improving documentation formatting and consistency


-or-

In .NET 11 and later versions, <paramref name="value" /> is greater than 0x7FFFFFC7 bytes.
Copy link

Copilot AI Jan 9, 2026

Choose a reason for hiding this comment

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

The exception documentation states "In .NET 11 and later versions, the value being set is greater than 0x7FFFFFC7 bytes" but doesn't have a period at the end of the sentence. According to documentation standards, sentences in comments should end with a period.

Copilot generated this review using guidance from repository custom instructions.

-or-

In .NET 11 and later versions, the value being set is greater than 0x7FFFFFC7 bytes.
Copy link

Copilot AI Jan 9, 2026

Choose a reason for hiding this comment

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

The exception documentation states "In .NET 11 and later versions" but doesn't have a period at the end of the sentence. According to documentation standards, sentences in comments should end with a period.

Copilot generated this review using guidance from repository custom instructions.
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.

This LGTM @gewarren

Let's :shipit:

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.

2 participants