Make linting green again for all files, not just lib 📗

.eslintrc

@@ -26,5 +26,13 @@
     "prefer-rest-params": "warn",
     "fp/no-loops": "warn"
   },
-  "overrides": []
+  "overrides": [
+    {
+      "files": [ "bin/*" ],
+      "rules": {
+        "no-console": "off",
+        "no-continue": "warn"
+      }
+    }
+  ]
 }

lib/client.js

@@ -88,27 +88,31 @@ DatadogMetricClient.prototype.request = function(method, path, params, callback)
   const req = https.request(http_options, function(res) {
     res.on('error', function(err) {
       if (typeof callback === 'function') {
-        callback(err, null, res.statusCode);
+        return callback(err, null, res.statusCode);
       }
     });
 
-    let data = '';
+    let _data = '';
     res.on('data', function(chunk) {
-      data += chunk;
+      _data += chunk;
     });
 
     res.on('end', function() {
       let error = null;
+      let data;
+
       try {
-        data = json.parse(data);
-      } catch (e) {}
+        data = json.parse(_data);
+      } catch (e) {
+        data = {};
+      }
       if (data.errors) {
         error = data.errors;
         data = null;
       }
 
       if (typeof callback === 'function') {
-        callback(error, data, res.statusCode);
+        return callback(error, data, res.statusCode);
       }
     });
   });
@@ -121,7 +125,7 @@ DatadogMetricClient.prototype.request = function(method, path, params, callback)
   // data being received, or SSL-related issues.
   req.on('error', function(err) {
     if (typeof callback === 'function') {
-      callback(err, null, 0);
+      return callback(err, null, 0);
     }
   });
 

test/api/embed.js

@@ -1,6 +1,5 @@
 const assert = require('assert');
 const querystring = require('querystring');
-const extend = require('extend');
 const sinon = require('sinon');
 const Client = require('../../lib/client');
 const Embed = require('../../lib/api/embed');
@@ -9,16 +8,17 @@ describe('api/embed', function() {
   const client = new Client({});
   const embed = Embed(client);
   let stub_request;
-  beforeEach(function() {
-    // Setup `client.request` as a stub
-    stub_request = sinon.stub(client, 'request');
-  });
-  afterEach(function() {
-    // Reset the original `client.request`
-    stub_request.restore();
-    stub_request = null;
-  });
+
   describe('#create', function() {
+    beforeEach(function() {
+      // Setup `client.request` as a stub
+      stub_request = sinon.stub(client, 'request');
+    });
+    afterEach(function() {
+      // Reset the original `client.request`
+      stub_request.restore();
+      stub_request = null;
+    });
     it('should make a valid api call', function() {
       const graphJSON = {
         viz: 'timeseries',

test/api/graph.js

@@ -1,8 +0,0 @@
-const assert = require('assert');
-const querystring = require('querystring');
-const extend = require('extend');
-const sinon = require('sinon');
-const client = require('../../lib/client');
-const graph = require('../../lib/api/graph');
-
-describe('api/graph', function() {});

test/api/metric.js

@@ -1,5 +1,4 @@
 const assert = require('assert');
-const extend = require('extend');
 const sinon = require('sinon');
 const Client = require('../../lib/client');
 const Metric = require('../../lib/api/metric');