Skip to content

Conversation

@Pr0methean
Copy link
Member

This replaces zip-rs/zip-old#434.

@Pr0methean Pr0methean enabled auto-merge May 1, 2024 17:00
@Pr0methean
Copy link
Member Author

@jmacadie Please sign your commits.

@Pr0methean Pr0methean changed the title doc: add delete/update examples doc(examples): add delete/update examples Jan 15, 2026
@Pr0methean Pr0methean added this to the 7.1.1 milestone Jan 15, 2026
@Its-Just-Nans
Copy link
Member

Hi

Why is there a process::exit in the examples ?

I'm not sure about the usefulness of it

I think the main() in examples could just return a Result<(), Box<dyn Error>> or something simple like that

Linked to #187

@Pr0methean Pr0methean modified the milestones: 7.1.1, 7.2.0 Jan 17, 2026
James MacAdie (TT sandbox) and others added 11 commits January 16, 2026 16:24
Per discussion #430, it was not obvious to me how to update a file
within an archive. I have added some examples of how to delete and
update to make it easier for anyone in a similar situation to me in the
future
… in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
… in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
… in path expression

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
@Pr0methean
Copy link
Member Author

@Its-Just-Nans Done in 73554df for the new examples; feel free to do so in a separate PR for the existing ones.

@Its-Just-Nans
Copy link
Member

@Its-Just-Nans Done in 73554df for the new examples; feel free to do so in a separate PR for the existing ones.

Okay I'm doing it

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.

3 participants