Skip to content
This repository has been archived by the owner on Nov 24, 2017. It is now read-only.

--minifyer not documented #28

Open
donpark opened this issue Jan 6, 2011 · 3 comments
Open

--minifyer not documented #28

donpark opened this issue Jan 6, 2011 · 3 comments

Comments

@donpark
Copy link

donpark commented Jan 6, 2011

following needs to be documented and included in command-line help as well:

  1. default minifier is yui_compressor.
    1.1 if yui_compressor is not installed, javascript merge will fail.
    1.2 if only closure_compiler is installed, javascript merge will fail.
  2. --minifyer (-m) option controls which minifier is used.
    valid values are 'yui_compressor' and 'closure_compiler'.

1.1 and 1.2 should be remedied so:

juicer defaults to whatever minifier is installed.

if more than one minifier is installed, either prompt user as well as informing them of -m option or go with default and inform user of -m option.

if no minifier is installed, prompt user to install either yui_compressor or closure_compiler.

@jamestomasino
Copy link

What about an option to not minify?

@cjohansen
Copy link
Owner

--minifyer ""

@jamestomasino
Copy link

Ahh, excellent! Thank you.

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

No branches or pull requests

3 participants