(refactor) rm vestigial comments + lint

Author: f3rno

README.md

@@ -4,15 +4,15 @@
 
 A Node.JS reference implementation of the Bitfinex API
 
-### Features
+## Features
 
 * Official implementation
 * REST v2 API
 * WebSockets v2 API
 * WebSockets v1 API
 
-
 ## Installation
+
 ```bash
   npm i --save bitfinex-api-node
 ```
@@ -35,6 +35,7 @@ Official API documentation at [https://docs.bitfinex.com/v2/reference](https://d
 ### Examples
 
 Sending an order & tracking status:
+
 ```js
 const ws = bfx.ws()
 
@@ -73,6 +74,7 @@ ws.open()
 ```
 
 Cancel all open orders
+
 ```js
 const ws = bfx.ws(2)
 
@@ -96,6 +98,7 @@ ws.open()
 ```
 
 Subscribe to trades by pair
+
 ```js
 const ws = bfx.ws(2)
 
@@ -114,9 +117,9 @@ ws.onTradeEntry({ symbol: 'tBTCUSD' }, (trades) => {
 ws.open()
 ```
 
-## Version 2.0.0 Breaking changes:
+## Version 2.0.0 Breaking changes
 
-### constructor takes only an options object now, including the API keys.
+### constructor takes only an options object now, including the API keys
 
 Old:
 

examples/rest2/order-history.js

@@ -16,7 +16,6 @@ const rest = bfx.rest(2, { transform: true })
 const pair = String(args[2])
 const symbol = pair[0] === 't' ? pair : `t${pair}`
 
-// TODO: natural lang start/end query args (a library must exist)
 const START = Date.now() - (30 * 24 * 60 * 60 * 1000 * 1000)
 const END = Date.now()
 const LIMIT = 25

examples/rest2/trade-history.js

@@ -16,7 +16,6 @@ const rest = bfx.rest(2, { transform: true })
 const pair = String(args[2])
 const symbol = pair[0] === 't' ? pair : `t${pair}`
 
-// TODO: natural lang start/end query args (a library must exist)
 const START = Date.now() - (30 * 24 * 60 * 60 * 1000 * 1000)
 const END = Date.now()
 const LIMIT = 25

index.js

@@ -48,14 +48,20 @@ class BFX {
     }
   }
 
+  /**
+   * Returns an arguments map ready to pass to a transport constructor
+   *
+   * @param {Object} extraOpts - options to pass to transport
+   */
   _getTransportPayload (extraOpts) {
-    return Object.assign({
+    return {
       apiKey: this._apiKey,
       apiSecret: this._apiSecret,
       authToken: this._authToken,
       company: this._company,
-      transform: this._transform
-    }, extraOpts)
+      transform: this._transform,
+      ...extraOpts
+    }
   }
 
   /**

lib/transports/ws2.js

@@ -481,7 +481,6 @@ class WSv2 extends EventEmitter {
    * @private
    */
   _onWSOpen () {
-    // TODO: Add _resetState() method for this, see _onWSClose
     this._isOpen = true
     this._isReconnecting = false
     this._packetWDLastTS = Date.now()
@@ -640,7 +639,7 @@ class WSv2 extends EventEmitter {
     }
 
     if (msg.length < 2) return
-    if (msg[1] === 'hb') return // TODO: optionally track seq
+    if (msg[1] === 'hb') return
 
     if (channelData.channel === 'book') {
       if (type === 'cs') {