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.

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

Minify JavaScript #517

Open
TrevorSayre opened this issue Dec 8, 2015 · 9 comments
Open

Minify JavaScript #517

TrevorSayre opened this issue Dec 8, 2015 · 9 comments

Comments

@TrevorSayre
Copy link

Currently Harp will minify your CoffeeScript, but if you write pure JavaScript files, then you're out of luck. I believe the same is true for Jade/HTML, LESS/CSS, etc. but I haven't tested. It would be nice to be able to pass an option to minify, or at least know an accepted process for Harp to minify your standard file types instead of relying on Grunt/Gulp/etc.

Since harp-minify is a thing, and it works just fine at minifying HTML/CSS/JavaScript, the above described behavior feels odd.

@ianrose
Copy link

ianrose commented Dec 8, 2015

@TrevorSayre This doesn't really answer your question, but my solution to this has been to use Gulp.js https://github.com/ianrose/storysettings/blob/master/gulpfile.js#L43-L47

@TrevorSayre
Copy link
Author

@ianrose I also use gulp to bring in browser-sync for dev and minify at compile time, but it would make sense for harp to do this itself. I posted this more as a suggestion than a question. Thank you for sharing, though!

@ianrose
Copy link

ianrose commented Dec 8, 2015

@TrevorSayre gotcha, agreed 👍

@cfjedimaster
Copy link

+1

@jeremyhaile
Copy link

This is especially true for me with vendor files that I have in the project unminified but would like minified in production.

@eltuctuc
Copy link

+1

@theofidry
Copy link

👍 this is an absolute necessity

@lroot
Copy link

lroot commented Jan 4, 2017

👍

1 similar comment
@kinduff
Copy link

kinduff commented Mar 4, 2018

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants