Skip to content

Conversation

@iremyux
Copy link
Contributor

@iremyux iremyux commented Nov 7, 2025

This PR updates the API documentation for TarFile and ZipFile ExtractToDirectory methods, clarifying the behavior of ZipFile.ExtractToDirectory and TarFile.ExtractToDirectory methods when extracting to pre-existing junctions or symbolic links in addition to e1c055b

Copilot AI review requested due to automatic review settings November 7, 2025 12:18
@iremyux iremyux requested a review from a team as a code owner November 7, 2025 12:18
@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 Nov 7, 2025
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 adds documentation clarifying the behavior when extracting archives to directories that are junctions or symbolic links. The documentation specifies that such links are followed and extraction writes to the final target folder.

  • Adds consistent documentation about junction/symbolic link handling during extraction
  • Updates affect both ZipFile and TarFile extraction methods

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
xml/System.IO.Compression/ZipFile.xml Adds seven instances of documentation clarifying junction/symbolic link behavior for various ZipFile.ExtractToDirectory overloads
xml/System.Formats.Tar/TarFile.xml Adds four instances of documentation clarifying junction/symbolic link behavior for various TarFile.ExtractToDirectory overloads

@iremyux iremyux requested a review from Copilot November 7, 2025 12:22
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

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

@iremyux iremyux requested a review from rokonec November 7, 2025 13:00
@iremyux iremyux merged commit aa655bd into dotnet:main Nov 7, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants