Release 1.0.8 (#11)

* version bump, lint updates & sync

* lint fixes

.eslintrc.json

@@ -0,0 +1,26 @@
+{
+  "env": {
+    "node": true,
+    "mocha": true
+  },
+  "extends": "eslint:recommended",
+  "installedESLint": true,
+  "root": true,
+  "rules": {
+    "comma-dangle": [2, "only-multiline"],
+    "dot-notation": 2,
+    "indent": [2, 4, {"SwitchCase": 1}],
+    "one-var": [2, "never"],
+    "no-trailing-spaces": [2, { "skipBlankLines": false }],
+    "keyword-spacing": [2, {
+      "before": true,
+      "after": true
+    }],
+    "no-delete-var": 2,
+    "no-empty": ["error", { "allowEmptyCatch": true }],
+    "no-label-var": 2,
+    "no-shadow": 2,
+    "no-unused-vars": [ 1, { "args": "none" }],
+    "no-console": 0
+  }
+}

config.js

@@ -95,7 +95,6 @@ Config.prototype.arrange_args = function (args) {
 
     for (var i=0; i < args.length; i++) {
         if (args[i] === undefined) continue;
-        var what_is_it = args[i];
         switch (typeof args[i]) {   // what is it?
             case 'function':
                 cb = args[i];

configfile.js

@@ -224,7 +224,7 @@ cfreader.ensure_enoent_timer = function () {
     cfreader._enoent_timer = setInterval(function () {
         var files = Object.keys(cfreader._enoent_files);
         for (var i=0; i<files.length; i++) {
-            var file = files[i];
+            var fileOuter = files[i];
             /* BLOCK SCOPE */
             (function (file) {
                 fs.stat(file, function (err) {
@@ -239,7 +239,7 @@ cfreader.ensure_enoent_timer = function () {
                         cfreader.on_watch_event(
                             file, args.type, args.options, args.cb));
                 });
-            })(file); // END BLOCK SCOPE
+            })(fileOuter); // END BLOCK SCOPE
         }
     }, 60 * 1000);
 };

package.json

@@ -3,7 +3,7 @@
   "name": "haraka-config",
   "license": "MIT",
   "description": "Haraka's config file loader",
-  "version": "1.0.7",
+  "version": "1.0.8",
   "homepage": "http://haraka.github.io",
   "repository": {
     "type": "git",
@@ -16,49 +16,12 @@
   "dependencies": {
     "js-yaml": "^3.5.3"
   },
-  "eslintConfig": {
-    "env": {
-      "node": true
-    },
-    "rules": {
-      "camelcase": [
-        0,
-        {
-          "properties": "never"
-        }
-      ],
-      "dot-notation": 2,
-      "indent": [
-        2,
-        4,
-        {
-          "SwitchCase": 1
-        }
-      ],
-      "one-var": [
-        2,
-        "never"
-      ],
-      "no-trailing-spaces": [
-        2,
-        {
-          "skipBlankLines": false
-        }
-      ],
-      "keyword-spacing": [
-        2,
-        {
-          "overrides": {}
-        }
-      ]
-    }
-  },
   "optionalDependencies": {},
   "devDependencies": {
-    "eslint": ">=2",
+    "eslint": "*",
     "grunt": "*",
     "grunt-eslint": "*",
-    "nodeunit": "^0.10.2"
+    "nodeunit": "*"
   },
   "bugs": {
     "mail": "helpme@gmail.com",

test/config.js

@@ -24,7 +24,7 @@ function setUp (done) {
     clearRequireCache();
     this.config = require('../config');
     done();
-};
+}
 
 exports.config = {
     'setUp' : setUp,