return es6 promise

[poying]

then we can do:

slack
  .webhook({
    channel: "#general",
    username: "webhookbot",
    text: "This is posted to #general and comes from a bot named webhookbot."
  })
  .then(function (res) {
    // ...
  })
  .catch(function (err) {
    // ...
  });

and

var res = yield slack.webhook({
  channel: "#general",
  username: "webhookbot",
  text: "This is posted to #general and comes from a bot named webhookbot."
});

[poying]

@clonn @tomchentw

There is a way to support es6 promise and we don't need to change the existing interface:

var callbackOrPromise = function (cb, fn) {
  if (cb || typeof Promise === 'undefined') {
    return fn(cb);
  }
  return new Promise(function (resolve, reject) {
    fn(function (err, res) {
      err ? reject(err) : resolve(res);
    });
  });
};

webhook: (options, callback) =>
  return callbackOrPromise
    callback
  , (callback) =>
      # ...

[tomchentw]

:+1:

[kevinburkeshyp]

hey, any update or further thoughts here?

[seawatts]

It says this was released in version 0.1.6 however I don't see it in there.

[georgerichmond]

+1

[kevinburkeshyp]

reading the commit history, I'm not sure this was implemented, unfortunately. gonna close this for now - feel free to implement/send a PR, or wrap via Promise.promisify in your own code base