Skip to content

[Draft] Move Format property from TarReader to TarEntry - #69950

Closed
carlossanlop wants to merge 3 commits into
dotnet:mainfrom
carlossanlop:TarFormat
Closed

[Draft] Move Format property from TarReader to TarEntry#69950
carlossanlop wants to merge 3 commits into
dotnet:mainfrom
carlossanlop:TarFormat

Conversation

@carlossanlop

Copy link
Copy Markdown
Contributor

Pending API proposal to approve: #69544

This PR addresses the proposed change.

@carlossanlopcarlossanlop added area-System.IO blocked Issue/PR is blocked on something - see comments labels May 28, 2022
@carlossanlopcarlossanlop added this to the 7.0.0 milestone May 28, 2022
@carlossanlopcarlossanlop self-assigned this May 28, 2022
@ghost

Copy link
Copy Markdown

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@ghost

Copy link
Copy Markdown

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

Issue Details

Pending API proposal to approve: #69544

This PR addresses the proposed change.

Author:carlossanlop
Assignees:carlossanlop
Labels:

area-System.IO, blocked

Milestone:7.0.0

@carlossanlop

Copy link
Copy Markdown
ContributorAuthor

Closing this in favor of #69933 .

@carlossanlop
carlossanlop deleted the TarFormat branch June 1, 2022 00:18
@ghostghost locked as resolved and limited conversation to collaborators Jul 1, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.IOblockedIssue/PR is blocked on something - see commentsnew-api-needs-documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@carlossanlop