enhancement: plop.load all assets

[RobinKnipe]

Due to a bug in the code, the load function doesn't load any plopfile assets when the config option is true.

The proposed change would allow

plop.load('plopfile', {}, true)

...as a shorthand for the following:

plop.load('plopfile', {}, {
    generators: true,
    helpers: true,
    partials: true,
    actionTypes: true
})

[RobinKnipe]

There is a PR to fix this plopjs/node-plop#187

[amwmedia]

According to the documentation, the include should always be an object. There are 4 valid properties for that object and "each of these properties should have an IncludeDefinition as its value."

[RobinKnipe]

@amwmedia I'm not sure what documentation you're reading, but this says otherwise: https://github.com/plopjs/plop/blob/master/plop-load.md#interface-includedefinition

[amwmedia]

That's the documentation I'm reading. It clearly needs to be updated, because the way it's written now is confusing. Above the IncludeDefinition interface description it reads: "include is an object that can contain 4 properties (generators, helpers, partials, and actionTypes). Each of these properties should have an IncludeDefinition as its value."

[RobinKnipe]

Hi @amwmedia - yes I see your point! You're right, I've misunderstood the Boolean part of that doc... Well in that case I guess this is a potential feature? I'll add a commit to update the documentation in line with my proposed change and then the maintainers can decide what they want to do about it... thanks!

[crutchcorn]

Given the quality of the contributions (code quality, tests added, documentation written, etc), I'd be happy to merge this (both PRs for node-plop and plop alike) in. @amwmedia, what do you think?

[crutchcorn]

Transfering this issue to https://github.com/plopjs/plop/ since we're migrating all of node-plop to that repo as a monorepo

[crutchcorn]

This has now been released as-of plop@3.1.0