gruntjs / grunt-docs

Grunt documentation. https://github.com/gruntjs/gruntjs.com
https://gruntjs.com/
79 stars 126 forks source link

Documentation - Configuring tasks is old #98

Closed Paolocargnin closed 10 years ago

Paolocargnin commented 10 years ago

HI,

Today I was studing for a new setup of my Gruntfile, and googling about uglify and concat i found this page: http://gruntjs.com/configuring-tasks

It tell to me to use both concat and uglify, but I know that uglify from a version that I don't know supports the concat. I think it's and old page but I'm not pretty sure, probably I missed something.

Instead, if i'm right, seemed right to call your attention because is a well indexed page on Google and I think that many people read it.

if I'm wrong because I don't understand the documentation and it explain other things that I can't figure out... Sorry. My bad.

Happy to lend a hand

By

leobalter commented 10 years ago

The concat and uglify tasks are just fine, but it's really not necessary to have the concat task if you just want to minify your js files with uglify, it also concat the files.

Anyway, that's not recommended, as with concat you can save one not minified file for development purposes and also the minified version of it with the uglify task, as seen on the configuring tasks page, like the example below:

dist: {
  src: 'src/<%= pkg.name %>.js',
  dest: 'dist/<%= pkg.name %>.min.js'
}

I'm closing this issue as it doesn't seem like a issue to be fixed on the documentation, but feel free to keep commenting and discuss about.