neo-async 2.0 is incompatible with 1.8 about neo-async

[suguru03]

https://github.com/caolan/async/blob/master/CHANGELOG.md#breaking-changes
filter, reject, some, every, and related functions now expect an error as the first callback argument, rather than just a simple boolean. Pass null as the first argument, or use fs.access instead of fs.exists. (#118, #774, #1028, #1041)

Async v2 and Neo-Async v2 were changed. The first argument always takes an error.
So you have to change as following.

function iterator(obj, done) {
  if (obj.id === 1) {
    done(null, true);
  } else {
    done(null, false);
  }
}

[cvlmtg]

then you have to fix the doc. I checked neo-async doc on http://suguru03.github.io/neo-async/doc/async.detect.html and it says:

// array (not use error handling)
var iterator = function(num, done) {
  setTimeout(function() {
    order.push(num);
    done(num % 2); // <------ here
  }, num * 10);
};

there is just one argument for the callback. You can't expect users of you library to read the documentation of another one.

[suguru03]

Oh, I'm sorry I forgot to fix them.
Thank you for pointing it out, I'll fix soon.

[cvlmtg]

thanks!