Merge pull request #414 from SassDoc/use-babel-eslint

Update to latest version of eslint and babel-eslint

.eslintrc

@@ -23,20 +23,19 @@ ecmaFeatures:
 
 rules:
   no-global-strict: 0
+  no-shadow: 0
   strict: 1
   no-undef: 1
   no-use-before-define: 0
   consistent-return: 0
-  comma-dangle: "always-multiline"
+  comma-dangle: [2, "always-multiline"]
   no-process-exit: 0
 
   quotes: [2, "single"]
   indent: [2, 2]
   no-mixed-spaces-and-tabs: [1, "smart-tabs"]
   no-spaced-func: 1
   space-after-keywords: [1, "always"]
-  space-before-function-parentheses: {"anonymous": "always", "named": "never"}
-  space-before-function-paren: {"anonymous": "always", "named": "never"}
-  space-before-blocks: "always"
-  space-in-brackets: "always"
+  space-before-function-paren: [2, {"anonymous": "always", "named": "never"}]
+  space-before-blocks: [1, "always"]
   no-underscore-dangle: 0

package.json

@@ -100,10 +100,10 @@
   },
   "devDependencies": {
     "babel": "5.5.*",
-    "babel-eslint": "^2.0.2",
+    "babel-eslint": "^3.1.15",
     "coveralls": "^2.11.2",
     "dateformat": "^1.0.11",
-    "eslint": "^0.18.0",
+    "eslint": "^0.23.0",
     "istanbul": "^0.3.13",
     "jsesc": "^0.5.0",
     "jshint": "^2.6.3",

src/annotation/annotations/index.js

@@ -18,5 +18,5 @@ module.exports = [
   require('./since.js'),
   require('./throw.js'),
   require('./todo.js'),
-  require('./type.js')
+  require('./type.js'),
 ];

src/annotation/annotations/name.js

@@ -15,6 +15,6 @@ export default function access() {
       }
     },
 
-    multiple: false
+    multiple: false,
   };
 }

src/annotation/annotations/require.js

@@ -192,7 +192,7 @@ function searchForMatches(code, regex, isAnnotatedByHandProxy, id = 1) {
   let matches = [];
 
   while ((match = regex.exec(code))) {
-    if (!isAnnotatedByHandProxy(match[id]) && (id <= 1 || match[id-1] === undefined)) {
+    if (!isAnnotatedByHandProxy(match[id]) && (id <= 1 || match[id - 1] === undefined)) {
       matches.push(match[id]);
     }
   }

src/annotation/annotations/see.js

@@ -9,7 +9,7 @@ export default function see(env) {
 
       let obj = {
         type: match[1] || 'function',
-        name: match[2]
+        name: match[2],
       };
 
       if (obj.name.indexOf('$') === 0) {

src/sassdoc.js

@@ -258,7 +258,7 @@ async function baseDocumentize(env) {
     recurse(),
     exclude(env.exclude || []),
     converter({ from: 'sass', to: 'scss' }),
-    filter
+    filter,
   ];
 
   let pipeline = () => {

test/annotations/envMock.js

@@ -3,5 +3,5 @@ module.exports = {
     warn: function () {},
     log: function () {},
     error: function () {},
-  }
+  },
 };

test/annotations/parameter.test.js

@@ -35,8 +35,8 @@ describe('#parameter', function () {
         warn: function (msg) {
           assert.equal(msg, '@parameter must at least have a name. Location: FileID:1:2');
           done();
-        }
-      }
+        },
+      },
     });
     assert.deepEqual(param.parse('{type} [default] - description\nmore\nthan\none\nline', { commentRange: { start: 1, end: 2 }}, 'FileID'), undefined);
   });

test/annotations/property.test.js

@@ -8,41 +8,41 @@ describe('#property', function () {
   it('should parse the prop annotation', function () {
     assert.deepEqual(prop.parse('base'), {
       type: 'Map',
-      name: 'base'
+      name: 'base',
     });
 
     assert.deepEqual(prop.parse('{Function} base.default'), {
       type: 'Function',
-      name: 'base.default'
+      name: 'base.default',
     });
 
     assert.deepEqual(prop.parse('{Function} base.default - description'), {
       type: 'Function',
       name: 'base.default',
-      description: 'description'
+      description: 'description',
     });
 
     assert.deepEqual(prop.parse('{Function} base.default [default] - description'), {
       type: 'Function',
       name: 'base.default',
       default: 'default',
-      description: 'description'
+      description: 'description',
     });
 
     assert.deepEqual(prop.parse('{Function} base.default [default] - description [with brackets]'), {
       type: 'Function',
       name: 'base.default',
       default: 'default',
-      description: 'description [with brackets]'
+      description: 'description [with brackets]',
     });
   });
 
-  it('should work for multiline description', function(){
+  it('should work for multiline description', function () {
     assert.deepEqual(prop.parse('{Function} base.default [default] - description\nmore\nthan\none\nline'), {
       type: 'Function',
       name: 'base.default',
       default: 'default',
-      description: 'description\nmore\nthan\none\nline'
+      description: 'description\nmore\nthan\none\nline',
     });
   });
 });

test/annotations/return.test.js

@@ -29,8 +29,8 @@ describe('#return', function () {
         warn: function (msg) {
           assert.equal(msg, '@return must at least have a type. Location: FileID:1:2');
           done();
-        }
-      }
+        },
+      },
     });
     assert.deepEqual(ret.parse('', { commentRange: { start: 1, end: 2 }}, 'FileID'), undefined);
   });

test/api/recurse.test.js

@@ -31,7 +31,7 @@ describe('#recurse', function () {
 describe('#recurse-null', function () {
   var files = [];
   var nullFile = new File({
-    contents: null
+    contents: null,
   });
 
   before(function (done) {

test/env/environment.test.js

@@ -107,8 +107,8 @@ describe('#environment', function () {
       display: {
         access: ['public', 'private'],
         alias: true,
-        watermark: true
-      }
+        watermark: true,
+      },
     };
 
     beforeEach(function () {

test/mock.js

@@ -32,7 +32,7 @@ Object.defineProperty(SassDocRc.prototype, 'contents', {
       throw new Error('SassDocRc.contents can only be an Object.');
     }
     this._contents = config;
-  }
+  },
 });
 
 module.exports.SassDocRc = SassDocRc;
@@ -52,7 +52,7 @@ inherits(MockLogger, Logger);
 Object.defineProperty(MockLogger.prototype, 'output', {
   get: function () {
     return this._output;
-  }
+  },
 });
 
 module.exports.Logger = MockLogger;