ci: add TravisCI and fix ESLint errors

Fixes: #7
PR-URL: #9
Reviewed-By: James Young <james.young@turner.com>

.eslintrc

@@ -20,6 +20,7 @@
         "keyword-spacing":             2,
         "linebreak-style":             [2, "unix"],
         "no-console":                  0,
+        "no-trailing-spaces":          2,
         "object-curly-spacing":        [2, "never"],
         "one-var":                     [2, "always"],
         "prefer-template":             2,

.travis.yml

@@ -0,0 +1,6 @@
+language: node_js
+node_js:
+    - "4.2"
+    - "4.3"
+    - "5.5"
+    - "5.6"

README.md

@@ -34,3 +34,11 @@ This package requires a few environment variable to be set.
 - `REGION`
 
 - `DEBUGMETRICS`
+
+
+
+[![build](https://img.shields.io/travis/cnnlabs/cnn-metrics/master.svg?style=flat-square)](https://travis-ci.org/cnnlabs/cnn-metrics)
+![node](https://img.shields.io/node/v/cnn-metrics.svg?style=flat-square)
+[![npm](https://img.shields.io/npm/v/cnn-metrics.svg?style=flat-square)](https://www.npmjs.com/package/cnn-metrics)
+[![npm-downloads](https://img.shields.io/npm/dm/cnn-metrics.svg?style=flat-square)](https://www.npmjs.com/package/cnn-metrics)
+[![dependency-status](https://gemnasium.com/cnnlabs/cnn-metrics.svg)](https://gemnasium.com/cnnlabs/cnn-metrics)

lib/graphite/client.js

@@ -23,16 +23,16 @@ Graphite.prototype.log = function (metrics) {
     debug(`Flush has been called ${this.flushCounter}`);
     this.flushCounter ++;
     if (this.flushCounter % 10 === 0) {          //show the message every 10 times
-        this.loggingMessageDisplayed = false; 
+        this.loggingMessageDisplayed = false;
     }
-    
+
     if (!this.loggingMessageDisplayed) {
         debug('Logging to Graphite is disabled by default on non-production environments. To enable is set ENVIRONMENT to "production". Or set DEBUGMETRICS=1 to debug metric counters');
         this.loggingMessageDisplayed = true;
-    };
-    
+    }
+
     // We don't want Graphite filling up with junk data from localhost
-    // so we disabled if logging isnt set up with valid data    
+    // so we disabled if logging isnt set up with valid data
     if (!this.isLogging ) {
         return;
     }

lib/metrics.js

@@ -20,7 +20,6 @@ Metrics = function (opts) {
     this.httpRequest = new Plugins.HttpRequest();
     this.httpResponse = new Plugins.HttpResponse();
 
-
     this.system = new Plugins.System();
     this.fetch = new Plugins.Fetch();
     this.aggregators = [];
@@ -36,17 +35,18 @@ Metrics.prototype.init = function (opts) {
     // Derive the keys based on the platform, Eg, <platform>.<application>.<instance>.<metric>
     const platform = (process.env.DYNO) ? 'heroku' : 'localhost',
         isProduction = (process.env.ENVIRONMENT === 'production' || process.env.NODE_ENV === 'production');
-       
+
 
     let self = this,
         instance,
         isLogging = (process.env.DEBUGMETRICS === '1');
-        
+
     isLogging = isLogging || isProduction;
-    
+
     if (!apiKey && (isLogging)) {
         throw 'No HOSTEDGRAPHITE_APIKEY is set. Please set a false one if you are on your localhost.';
     }
+
     //Lets do something with Kub here...
     if (platform === 'heroku') {
         instance = process.env.DYNO.replace('.', '_');
@@ -177,7 +177,7 @@ Metrics.prototype.instrument = function (obj, opts) {
         case 'http.request':
             this.httpRequest.instrument(obj);
             break;
-            
+
         default:
             throw new Error('No valid "opts.as" argument given. You need to specify the object you want instrumented.');
     }