Skip to content

Add Folders to ZipArchive#342

Open
dos-ise wants to merge 3 commits intoadamhathcock:masterfrom
dos-ise:AddFolders
Open

Add Folders to ZipArchive#342
dos-ise wants to merge 3 commits intoadamhathcock:masterfrom
dos-ise:AddFolders

Conversation

@dos-ise
Copy link
Copy Markdown

@dos-ise dos-ise commented Jan 18, 2018

Add Folders to ZipArchive like in dotnetzip. Not yet ready

Copy link
Copy Markdown
Owner

@adamhathcock adamhathcock left a comment

Choose a reason for hiding this comment

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

Why so this over using the existing extension methods?

@dos-ise
Copy link
Copy Markdown
Author

dos-ise commented Jan 18, 2018

You mean the AddAllFromDirectory Extension?
This should create an empty folder in the zip file instead adding an folder and all of its contetn.

Its a behavior from dotnetzip we use quit alot.

@adamhathcock
Copy link
Copy Markdown
Owner

I couldn't quite tell what you were going for. So you want the ability to add directory entries?

@dos-ise
Copy link
Copy Markdown
Author

dos-ise commented Jan 19, 2018

Correct, I added a test to clarify for what I am trying to achieve. Sorry for the misleading commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants