Skip to content

Conversation

gewarren
Copy link
Contributor

@gewarren gewarren commented Oct 3, 2025

The exceptions refer to parameters that aren't part of the methods.
(Hide whitespace changes.)

@gewarren gewarren requested a review from a team as a code owner October 3, 2025 18:41
@Copilot Copilot AI review requested due to automatic review settings October 3, 2025 18:41
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 fixes XML documentation by removing exception elements that reference non-existent parameters and correcting parameter references. The changes ensure that exception documentation accurately matches the actual method signatures.

  • Removes exception elements that reference parameters not present in the method signatures
  • Corrects parameter name casing in exception documentation
  • Consolidates duplicate exception conditions into single elements

Reviewed Changes

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

File Description
xml/System.IO/DirectoryInfo.xml Removes exception referencing non-existent path parameter
xml/System.IO.Log/ReservationCollection.xml Removes exceptions referencing non-existent archiveTail and size parameters
xml/System.IO.Log/LogStore.xml Corrects parameter name casing from Mode to mode and adds proper see references
xml/System.IO.Log/LogExtentCollection.xml Removes exception for non-existent archiveTail parameter and consolidates size exception text

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 3, 2025
@gewarren gewarren enabled auto-merge (squash) October 3, 2025 18:42
Copy link
Contributor

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

@dotnet-policy-service dotnet-policy-service bot removed the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 3, 2025
@gewarren gewarren requested a review from BillWagner October 3, 2025 22:26
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.

1 participant