gruntjs.com icon indicating copy to clipboard operation
gruntjs.com copied to clipboard

`grunt.file` documents unreleased feature: grunt.file.copy options "process"

Open Sulley-Bear opened this issue 10 years ago • 4 comments

Originally report: Issue # 1440 (gruntjs/grunt)

grunt.file.md advises that for the grunt.file.copy function:

  // The source file contents, source file path, and destination file path 
  // are passed into this function, whose return value will be used as the
  // destination file's contents. If this function returns `false`, the file
  // copy will be aborted.
  process: processFunction,

In particular: "The ... destination file path [is] passed into this function"

As of the latest release of Grunt, 0.4.5, this is not true. The documentation needs to be corrected to remove this, or advise that it will be available in future releases.

Sulley-Bear avatar Feb 04 '16 09:02 Sulley-Bear

Hi @JKAussieSkater. Are you working on this issue? If not, I'd like to submit a PR in the next few days. Let me know what you think.

AurelioDeRosa avatar Apr 18 '16 00:04 AurelioDeRosa

Please go for it! I was hoping to get it done before I went back to university/college, but I couldn't quite squeeze it in!

Sulley-Bear avatar Apr 18 '16 00:04 Sulley-Bear

Do you mind assign it to me as a reminder?

AurelioDeRosa avatar Apr 18 '16 00:04 AurelioDeRosa

Per https://github.com/gruntjs/gruntjs.com/issues/233, I suggest we improve it by stating that it was "Added in vX" with the release where it was added.

Krinkle avatar Sep 10 '24 03:09 Krinkle