vkadam / grunt-jsbeautifier

Beautify js, css, html and json files using Grunt and https://github.com/einars/js-beautify

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Allow to write to `dest`

twolfson opened this issue · comments

Most grunt plugins do not modify a file in-place. Can you add a dest option to output the beautified content?

{
  src: 'abc.min.js',
  dest: 'abc.js'
}

Added support in 0.2.5 version.

This was not properly implemented. grunt has dest as a built in property for tasks.

http://gruntjs.com/configuring-tasks#files

I don't think it will conflict, as "dest" is not on root object, its options.dest

{
  src: 'abc.min.js',
  options: {
    dest: "dest"
  }
}

Also please not "dest" will be path of folder and not file name.
I have bumped up version to 0.2.6.(Renamed dist to dest, thanks to you)

I am stating that you are not following grunt conventions. It is expected that the src files will be written to dest when the task completes. dest is expected to be a path to a file, not a folder.

If you want to build a batch grunt task, then that should be something separate (e.g. jsbeautifier-dir).