Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

updating archiver dependency #32

Closed
wants to merge 1 commit into from
Closed

updating archiver dependency #32

wants to merge 1 commit into from

Conversation

maxmil7
Copy link

@maxmil7 maxmil7 commented Sep 27, 2016

Fix for #31
Tested it locally. I was able to download the zip file w/o any issues.

@shaunwarman
Copy link

👍 @gotwarlost

https://github.com/archiverjs/node-archiver#usage

@linkRace
Copy link

linkRace commented Oct 3, 2016

Also leaving a 👍 on this, is this repo still being maintained?

@mcollina
Copy link
Contributor

mcollina commented Nov 6, 2016

@gotwarlost?

@grawk grawk mentioned this pull request Feb 13, 2017
Open
@nagapavan
Copy link

nagapavan commented Sep 15, 2017 •
edited
Loading

Any update on when this PR will be merged?
This will address the Security issue "https://nodesecurity.io/advisories/118", with Minimatch.

@maxmil7
Copy link
Author

maxmil7 commented Nov 30, 2018

Closing this in favor of #40

@maxmil7 maxmil7 closed this Nov 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@maxmil7 @shaunwarman @linkRace @mcollina @nagapavan