Upgrade bignumber.js to latest version. (#309)

* Prepare for bignumber.js 6.0.0: cmp() alias was removed. * Prepare for bignumber.js 6.0.0: sub() alias was removed. * Upgrade bignumber.js to latest version.
coinbase · Apr 18, 2018 · f89bbf9 · f89bbf9
1 parent 5f2a11f
commit f89bbf9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/orderbook.js b/lib/orderbook.js
@@ -5,8 +5,8 @@ const assert = require('assert');
 class Orderbook {
   constructor() {
     this._ordersByID = {};
-    this._bids = new RBTree((a, b) => a.price.cmp(b.price));
-    this._asks = new RBTree((a, b) => a.price.cmp(b.price));
+    this._bids = new RBTree((a, b) => a.price.comparedTo(b.price));
+    this._asks = new RBTree((a, b) => a.price.comparedTo(b.price));
   }
 
   _getTree(side) {
@@ -99,7 +99,7 @@ class Orderbook {
 
     assert(order);
 
-    order.size = order.size.sub(size);
+    order.size = order.size.minus(size);
     this._ordersByID[order.id] = order;
 
     assert(order.size >= 0);

diff --git a/package.json b/package.json
@@ -22,7 +22,7 @@
   ],
   "dependencies": {
     "@types/request": "2.47.0",
-    "bignumber.js": "^5.0.0",
+    "bignumber.js": "^6.0.0",
     "bintrees": "^1.0.1",
     "request": "^2.81.0",
     "ws": "^4.0.0"