Add Promise compatibility

README.md

@@ -35,6 +35,10 @@ t.get("/1/members/me", { cards: "open" }, function(err, data) {
   if (err) throw err;
   console.log(data);
 });
+
+// Use it as a promise
+t.get("/1/members/me", { cards: "open" })
+  .then(function(data) { console.log(data); });
 ```
 
 ### Uploading attachments to a card

lib/node-trello.js

@@ -83,15 +83,22 @@ Trello.prototype.request = function (method, uri, argsOrCallback, callback) {
     options.json = this.addAuthArgs(this.parseQuery(uri, args));
   }
 
-  request[method === 'DELETE' ? 'del' : method.toLowerCase()](options, function (err, response, body) {
-    if (!err && response.statusCode >= 400) {
-      err = new Error(body);
-      err.statusCode = response.statusCode;
-      err.responseBody = body;
-      err.statusMessage = require('http').STATUS_CODES[response.statusCode];
-    }
-
-    callback(err, body);
+  return new Promise(function (resolve, reject) {
+    request[method === 'DELETE' ? 'del' : method.toLowerCase()](options, function (err, response, body) {
+      if (!err && response.statusCode >= 400) {
+        err = new Error(body);
+        err.statusCode = response.statusCode;
+        err.responseBody = body;
+        err.statusMessage = require('http').STATUS_CODES[response.statusCode];
+      }
+
+      if (typeof callback === 'function') {
+        callback(err, body);
+      } else {
+        if (err) reject(err);
+        else resolve(body)
+      }
+    });
   });
 };
 

package.json

@@ -22,7 +22,7 @@
   "devDependencies": {
     "blanket": "^1.2.3",
     "mocha": "^3.4.2",
-    "should": "^5.2.0",
+    "should": "^12.0.0",
     "travis-cov": "^0.2.5"
   },
   "config": {

test/trello.js

@@ -209,6 +209,11 @@ describe("Trello", function () {
 
         request.get = method;
       });
+
+      it("should return a promise", function () {
+        const promise = this.trello.request("VERB", "/test");
+        promise.should.be.Promise();
+      });
     });
   });
 });