zoom icon indicating copy to clipboard operation
zoom copied to clipboard

[enhancement] What about a build script

Open ppowalowski opened this issue 9 years ago • 2 comments

The repo contains a minimized version. There is no automated buildscript to build a minimized version - If contributors change the source of jquery.zoom.js there is no predefined way to build the ´ jquery.zoom.min.js´ file. Either every contributor finds a way to minimize the file(alowing inconsistent results) or they won't build the code at all so PRs will become unmergable without intervention causing longer developemt cycles.

I suggest gulp or something morge minimalistic like the usage of npm itself for building

ppowalowski avatar Jun 01 '15 12:06 ppowalowski

I ran into the same question. See my PR: https://github.com/jackmoore/zoom/pull/100/files#diff-0730bb7c2e8f9ea2438b52e419dd86c9R5

tuomassalo avatar Feb 08 '16 09:02 tuomassalo

Thanks, and sorry about the lack of a build script. I keep a private build script that did the minification, lint, bumped the bower and package json files, and published to npm since contributors wouldn't be concerned with most of that. I don't actually want pull requests with minified files, just to be on the safe side. But, I should separate the source from the versioned distribution files and add a build process.

jackmoore avatar Feb 08 '16 18:02 jackmoore