tree() returns an array about undertaker HOT 9 CLOSED

yea, pass {deep: true} to have json tree like

[ { label: 'should throw TypeError when `name` is not a string',
    type: 'task',
    nodes: [] },
  { label: 'should throw TypeError when `callback` is not a function',
    type: 'task',
    nodes: [] },
  { label: 'should return false if given `name` exist in npm registry',
    type: 'task',
    nodes: [] },
  { label: 'should return true if given `name` not exist in npm registry',
    type: 'task',
    nodes: [] },
  { label: 'should return false when repository already exists',
    type: 'task',
    nodes: [] },
  { label: 'should return true when repository not exists',
    type: 'task',
    nodes: [] } ]

this is copy-pasta from my test runner based on undertaker.

from undertaker.

@tunnckoCore Thanks, I missed that. I saw the documentation about deep and tried passing a bool directly rather than an object with the deep property.

Anyway, what an intuitive API. Ask for a tree and get something else.

from undertaker.

@mikehaas763 no need to be a sarcastic asshole about it. I actually agree that it is not intuitive and I am looking into switching it over to a tree. It seems like I can make it work with gulp's CLI for 4.0 without too much trouble.

from undertaker.

@phated maybe its a good idea Undertaker to have also .tasks() method which will just return array with task names. and tree to be just as .tree({deep: true})

from undertaker.

@tunnckoCore that API is too close to task, the main API, and I know it would cause more issues than it would solve.

from undertaker.

hmm.. nope, it would be intuitive.

• .task(name) - get a task with specified name
• .tasks() - get all tasks
• .tree() - archey list

there's no place for confusion.

from undertaker.

there is plenty of room for confusion when someone typos task as tasks. I've already considered this and am not going to do that

from undertaker.

@mikehaas763 be aware that I am looking into this, but I am going to lock this for my sanity.

from undertaker.

Completed in c1cda41

from undertaker.