lint: use standard style

Author: dougwilson

.eslintignore

@@ -0,0 +1,2 @@
+coverage
+node_modules

.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "standard"
+}

.travis.yml

@@ -14,7 +14,7 @@ cache:
     - node_modules
 before_install:
   # Setup Node.js version-specific dependencies
-  - "test $TRAVIS_NODE_VERSION != '0.8' || npm rm --save-dev istanbul"
+  - "test $TRAVIS_NODE_VERSION != '0.8' || npm rm --save-dev eslint eslint-config-standard eslint-plugin-promise eslint-plugin-standard istanbul"
 
   # Update Node.js modules
   - "test ! -d node_modules || npm prune"
@@ -23,5 +23,6 @@ script:
   # Run test script, depending on istanbul install
   - "test ! -z $(npm -ps ls istanbul) || npm test"
   - "test   -z $(npm -ps ls istanbul) || npm run-script test-travis"
+  - "test   -z $(npm -ps ls eslint  ) || npm run-script lint"
 after_script:
   - "test -e ./coverage/lcov.info && npm install coveralls@2 && cat ./coverage/lcov.info | coveralls"

index.js

@@ -44,7 +44,7 @@ var cacheControlNoTransformRegExp = /(?:^|,)\s*?no-transform\s*?(?:,|$)/
  * @public
  */
 
-function compression(options) {
+function compression (options) {
   var opts = options || {}
 
   // options
@@ -55,25 +55,26 @@ function compression(options) {
     threshold = 1024
   }
 
-  return function compression(req, res, next){
+  return function compression (req, res, next) {
     var ended = false
     var length
     var listeners = []
-    var write = res.write
-    var on = res.on
-    var end = res.end
     var stream
 
+    var _end = res.end
+    var _on = res.on
+    var _write = res.write
+
     // flush
-    res.flush = function flush() {
+    res.flush = function flush () {
       if (stream) {
         stream.flush()
       }
     }
 
     // proxy
 
-    res.write = function(chunk, encoding){
+    res.write = function write (chunk, encoding) {
       if (ended) {
         return false
       }
@@ -84,10 +85,10 @@ function compression(options) {
 
       return stream
         ? stream.write(new Buffer(chunk, encoding))
-        : write.call(this, chunk, encoding)
-    };
+        : _write.call(this, chunk, encoding)
+    }
 
-    res.end = function(chunk, encoding){
+    res.end = function end (chunk, encoding) {
       if (ended) {
         return false
       }
@@ -102,7 +103,7 @@ function compression(options) {
       }
 
       if (!stream) {
-        return end.call(this, chunk, encoding)
+        return _end.call(this, chunk, encoding)
       }
 
       // mark ended
@@ -112,11 +113,11 @@ function compression(options) {
       return chunk
         ? stream.end(new Buffer(chunk, encoding))
         : stream.end()
-    };
+    }
 
-    res.on = function(type, listener){
+    res.on = function on (type, listener) {
       if (!listeners || type !== 'drain') {
-        return on.call(this, type, listener)
+        return _on.call(this, type, listener)
       }
 
       if (stream) {
@@ -129,13 +130,13 @@ function compression(options) {
       return this
     }
 
-    function nocompress(msg) {
+    function nocompress (msg) {
       debug('no compression: %s', msg)
-      addListeners(res, on, listeners)
+      addListeners(res, _on, listeners)
       listeners = null
     }
 
-    onHeaders(res, function(){
+    onHeaders(res, function onResponseHeaders () {
       // determine if request is filtered
       if (!filter(req, res)) {
         nocompress('filtered')
@@ -157,16 +158,16 @@ function compression(options) {
         return
       }
 
-      var encoding = res.getHeader('Content-Encoding') || 'identity';
+      var encoding = res.getHeader('Content-Encoding') || 'identity'
 
       // already encoded
-      if ('identity' !== encoding) {
+      if (encoding !== 'identity') {
         nocompress('already encoded')
         return
       }
 
       // head
-      if ('HEAD' === req.method) {
+      if (req.method === 'HEAD') {
         nocompress('HEAD request')
         return
       }
@@ -196,35 +197,35 @@ function compression(options) {
       addListeners(stream, stream.on, listeners)
 
       // header fields
-      res.setHeader('Content-Encoding', method);
-      res.removeHeader('Content-Length');
+      res.setHeader('Content-Encoding', method)
+      res.removeHeader('Content-Length')
 
       // compression
-      stream.on('data', function(chunk){
-        if (write.call(res, chunk) === false) {
+      stream.on('data', function onStreamData (chunk) {
+        if (_write.call(res, chunk) === false) {
           stream.pause()
         }
-      });
+      })
 
-      stream.on('end', function(){
-        end.call(res);
-      });
+      stream.on('end', function onStreamEnd () {
+        _end.call(res)
+      })
 
-      on.call(res, 'drain', function() {
+      _on.call(res, 'drain', function onResponseDrain () {
         stream.resume()
-      });
-    });
+      })
+    })
 
-    next();
-  };
+    next()
+  }
 }
 
 /**
  * Add bufferred listeners to stream
  * @private
  */
 
-function addListeners(stream, on, listeners) {
+function addListeners (stream, on, listeners) {
   for (var i = 0; i < listeners.length; i++) {
     on.apply(stream, listeners[i])
   }
@@ -234,7 +235,7 @@ function addListeners(stream, on, listeners) {
  * Get the length of a given chunk
  */
 
-function chunkLength(chunk, encoding) {
+function chunkLength (chunk, encoding) {
   if (!chunk) {
     return 0
   }
@@ -249,7 +250,7 @@ function chunkLength(chunk, encoding) {
  * @private
  */
 
-function shouldCompress(req, res) {
+function shouldCompress (req, res) {
   var type = res.getHeader('Content-Type')
 
   if (type === undefined || !compressible(type)) {
@@ -265,11 +266,11 @@ function shouldCompress(req, res) {
  * @private
  */
 
-function shouldTransform(req, res) {
+function shouldTransform (req, res) {
   var cacheControl = res.getHeader('Cache-Control')
 
   // Don't compress for Cache-Control: no-transform
   // https://tools.ietf.org/html/rfc7234#section-5.2.2.4
-  return !cacheControl
-    || !cacheControlNoTransformRegExp.test(cacheControl)
+  return !cacheControl ||
+    !cacheControlNoTransformRegExp.test(cacheControl)
 }

package.json

@@ -17,6 +17,10 @@
     "vary": "~1.1.0"
   },
   "devDependencies": {
+    "eslint": "2.9.0",
+    "eslint-config-standard": "5.3.1",
+    "eslint-plugin-promise": "1.1.0",
+    "eslint-plugin-standard": "1.3.2",
     "istanbul": "0.4.3",
     "mocha": "2.4.5",
     "supertest": "1.1.0"
@@ -30,6 +34,7 @@
     "node": ">= 0.8.0"
   },
   "scripts": {
+    "lint": "eslint **/*.js",
     "test": "mocha --check-leaks --reporter spec --bail",
     "test-cov": "istanbul cover node_modules/mocha/bin/_mocha -- --check-leaks --reporter dot",
     "test-travis": "istanbul cover node_modules/mocha/bin/_mocha --report lcovonly -- --check-leaks --reporter spec"

test/.eslintrc

@@ -0,0 +1,5 @@
+{
+  "env": {
+	  "mocha": true
+  }
+}