Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

grunt.file documents unreleased feature: grunt.file.copy options "process" #236

Open
JKVeganAbroad opened this issue Feb 4, 2016 · 4 comments

Comments

@JKVeganAbroad
Copy link
Contributor

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.

@JKVeganAbroad JKVeganAbroad self-assigned this Feb 4, 2016
@AurelioDeRosa
Copy link

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.

@JKVeganAbroad
Copy link
Contributor Author

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!

@AurelioDeRosa
Copy link

Do you mind assign it to me as a reminder?

@Krinkle Krinkle transferred this issue from gruntjs/grunt-docs Sep 10, 2024
@Krinkle Krinkle changed the title grunt.file.md documents unreleased feature: grunt.file.copy options "process" grunt.file documents unreleased feature: grunt.file.copy options "process" Sep 10, 2024
@Krinkle
Copy link
Contributor

Krinkle commented Sep 10, 2024

Per #233, I suggest we improve it by stating that it was "Added in vX" with the release where it was added.

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

No branches or pull requests

3 participants