node-zip icon indicating copy to clipboard operation
node-zip copied to clipboard

Please avoid using global object for exporting any stutf

Open escaton opened this issue 9 years ago • 2 comments

escaton avatar Feb 07 '16 14:02 escaton

I made this repository some years ago, when there wasn't any Node.js zip library. I found the project JSZip, they made a great job porting zip/unzip archiving functionality, but it used to work only on browsers. That's why I made this repo, but now things have changed and JSZip I think it works with node, but this module is used at many projects (I didn't expect this to be honest).

Accepting your pull request could potentially break many dependent projects that rely on this functionality (we could think on a version bump maybe).

I would rather wait a bit and ask to see if more people are okay with the change.

daraosn avatar Feb 07 '16 20:02 daraosn

just bump the big version number and projects should carefully upgrade. breaking api shouldnt be a problem if proper documentation is provided, and this change is trivial

trufae avatar Sep 13 '16 21:09 trufae