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

CSS/JS Minification #32

Open
jchannon opened this issue Aug 10, 2013 · 7 comments
Open

CSS/JS Minification #32

jchannon opened this issue Aug 10, 2013 · 7 comments
Assignees
Milestone

Comments

@jchannon
Copy link
Member

How about the ability to minify specified JS/CSS files/folders?

@phillip-haydon
Copy link
Member

Being a static site I don't believe there's any real benefit, once you've designed your site, you minify once and you're done. I think people should just minify themselves, there's heaps of sites they can do it on.

@jchannon
Copy link
Member Author

Just thought it would make a nice automation feature. Snow runs,
produces output and minifies. All you have to do is deploy

@phillip-haydon
Copy link
Member

I think we should leave this for now. Maybe look at it for V2

@ghost ghost assigned jchannon Aug 12, 2013
@phillip-haydon
Copy link
Member

I put this as milestone 2.

Don't start this until 1 is released.

If you look in my Snow folder, I have an image optimization library in there, we should couple that into the same scenario, basically allow people to create a batch file which can be executed and does image optimization, minification, bundling, etc.

@luisrudge
Copy link

Well... May I chime in? I think that this is a useful feature for many scenarios.. I could have many scripts and i don't want to run all of them through a site to minify.. The css can change from time to time because of new things i'm doing with my posts etc.. And it's easy to tackle that. :)

@horsdal
Copy link
Contributor

horsdal commented Oct 9, 2014

+1 for adding minification. Had been trying to get around to doing a PR with this.

I would want it disabled when the debug flag is set on the cli.

@mwhelan
Copy link

mwhelan commented Oct 10, 2014

+1. I agree this would be quite useful.

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

No branches or pull requests

5 participants