@sveinburne I'm not so keen on github dependencies. Will you be supporting unzip2's replacement, if so, could you publish to npm?

@guyonroche Not planning on supporting a replacement, sorry :-(

If glebdmitriew/node-unzip-2#21 is merged any time soon, there won't be a need for this PR 🎉 🤞

PS: I'm also here from this yarn warning:

warning exceljs > unzip2 > fstream > graceful-fs@3.0.11: please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js

Good news – there is a new release on npm!

It somehow happened that unzip2's repo on github (https://github.com/glebdmitriew/node-unzip-2) was published to npm by someone else, not the author. See https://www.npmjs.com/package/unzip2.

A couple of hours ago the author of the github repo published the library as https://www.npmjs.com/package/node-unzip-2 and repointed README to node-unzip-2. Looks like node-unzip-2 has existed for a while, but was not mentioned from the github repo.

So here's what should work:

npm rm unzip2
npm install node-unzip-2

require / import statements will need an update too.

@jsamr would you be happy to submit a PR for that?

@kachkaev - massive thanks for doing all this legwork!
If that's all it takes (I've just run the tests and it seems so) then I can push your suggested change now

That'd be great! Thanks for this truly awesome library!

Closing as fixed in master. Released as 0.8.3

I confirm no more warnings from exceljs after upgrading! 🎉

@jsamr thanks for your efforts to resolve this! I probably would not start digging though github repos and npm packages if you did not submit this PR!