Programic API

Desperate need for documentation here.

I have been an advocate for npm scripting of build processes for many departments at my work and I sold them a group of npm packages to get the job done, copyfiles being one of them. We have recently converted from npm scripts within package.json to writing out a node build file.

We are going to have to switch away from copyfiles do to the lack of documentation and below.

takes an array of paths, last one is the destination path…

On the command line that makes sense, programatically it is nonsensical. I can’t explain to other developers why the “last item in the array is the destination” and any new developer coming on board would have to get taught that as well.

This npm package has been great to me, I would love to see more documentation on “programmatic” usage as well an additional parameter when calling copyfiles in that environment.

Something like:

---

copyfiles(path, dest [, opts] [, callback])

Example

var copyfiles = require('copyfiles');

copyfiles(["./src/**/*.html"], "./dist/", function (err) {
    if (err) console.error(err);
});

Example w/opts

var copyfiles = require('copyfiles');

// I honestly don't understand how you are taking options here because it isn't documented.
copyfiles(["./src/**/*.html"], "./dist/", "-u", function (err) {
    if (err) console.error(err);
});

[spenoir]

Actually in my case where I wanted to flatten while copying Programmatically I had to do this:

copyfiles(["./src/**/*.html", "./dist/"], true, function (err) {
    if (err) console.error(err);
});

I found this out by checking your tests. This really needs to be documented

[topalavlad]

I know this issue is pretty old but using true as the second parameter stops you from adding other parameters like error or verbose.

After looking through the code I saw that flat = true is actually transformed into up = true so for anyone who wants to use multiple parameters one of which is flat, the following will do the trick:

copyfiles(["./src/**/*.html", "./dist/"], {up: true, error: true, verbose: true}, function (err) {
    if (err) console.error(err);
});

@calvinmetcalf might be worth allowing people to use flat in the parameters object since the Read.me says:

if it is an object it is a hash of the various options (the long version e.g. up, all, flat, exclude, error, verbose and soft)

If you agree, I can create a PR.