search

eduardolundgren / gulp-umd

Gulp plugin for build JavaScript files as Universal Module Definition, aka UMD
MIT License
126 stars 14 forks source link

Does this package need the `gulp` dependency? #19

Closed ChALkeR closed 7 years ago

ChALkeR commented 8 years ago

It doesn't seem to be require-d outside of tests.

Perhaps you should move it to devDependencies and/or peerDependencies.

CodeMan99 commented 7 years ago

Same seems to be true of nodeunit. This package is huge and has lots of child dependencies.

CodeMan99 commented 7 years ago

Created PR #20.

eduardolundgren commented 7 years ago

Merged. Thank you.

CodeMan99 commented 7 years ago

@eduardolundgren Would you please publish a new version to npmjs? Needing to install the (now) devDependencies in production is eating up a heap of production server disk space. Sadly, this is a pretty large problem in my case.

eduardolundgren commented 7 years ago

done.