Skip to content

[artifact] replace unzipper with unzip-stream#1602

Merged
robherley merged 3 commits intomainfrom
robherley/replace-unzip-lib
Dec 11, 2023
Merged

[artifact] replace unzipper with unzip-stream#1602
robherley merged 3 commits intomainfrom
robherley/replace-unzip-lib

Conversation

@robherley
Copy link
Copy Markdown
Contributor

Unfortunately the unzipper library we first chose has some issues with node > 18:

So we're replacing with unzip-stream. In the future we may want to invest more changes into this, and keep track of a more maintained unzip library. Unfortunately there aren't a lot of great options that also stream the contents without writing the entire archive to disk:

Closes:

Example run:

@robherley robherley requested a review from a team as a code owner December 11, 2023 17:22
@robherley robherley requested a review from jtamsut December 11, 2023 19:05
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