Minify option on build about component HOT 6 CLOSED

Should be an option, could use uglifyJS

from component.

-1 from me becauuuuse this would typically be deployment thing, which is super ad-hoc

from component.

Just tried to add the option usign UglifyJS. It's super easy but I'm agree is more for deployment than dev.

I'll close the ticket. If anyone wants to minify the build.js can just $ component build && uglifyjs -o build/build.js (if you build into the default path)

from component.

not sure i get this ... when i use uglify on a build.js i only minify the outer loader script of component.io, i.e. the require functionality. uglify does not minify the scripts inside of the require statements since they are just strings from the perspective of uglify... but minimizing has to run during the gathering phase of the component.io builder ..

thats where https://github.com/Wizcorp/component-uglifyjs and other build step components come in .. just thought i should mention it here, in case others find this issue and think just passing the build.js in would be enough

from component.

You should build without the --dev option then. See https://github.com/component/builder.js/blob/master/lib/plugins/commonjs.js#L65-L70

from component.

Awesome .. It didn't know it did that .. I had thought dev would only add the source file names irrespective of the content...

Doing it as part of the pipeline though might have the advantage of still knowing where a potential error originated from .
but that's not really a good reason ... And it might not work anyway..

-----Original Message-----
From: "Julian Gruber" [email protected]
Sent: ‎30.‎01.‎2014 09:38
To: "component/component" [email protected]
Cc: "matthiasg" [email protected]
Subject: Re: [component] Minify option on build (#50)

You should build without the --dev option then. See https://github.com/component/builder.js/blob/master/lib/plugins/commonjs.js#L65-L70
—
Reply to this email directly or view it on GitHub.

from component.