Simpler and stricter eslint

Author: tmcw

.eslintrc

@@ -33,6 +33,7 @@
     "no-shadow": 0,
     "no-undef": 2
   },
+  "extends": "eslint:recommended",
   "env": {
     "node": true,
     "es6": true,

bin/documentation.js

@@ -1,5 +1,7 @@
 #!/usr/bin/env node
 
+/* eslint no-console: 0 */
+
 'use strict';
 
 var documentation = require('../'),

lib/infer/params.js

@@ -13,25 +13,29 @@ var types = require('ast-types'),
  */
 module.exports = function inferParams(comment) {
 
-  function paramToDoc(param) {
-    // ES6 default
-    if (param.type === 'AssignmentPattern') {
-      var newParam = paramToDoc(param.left);
-      var optionalParam = {
-        title: 'param',
-        name: newParam.name,
-        'default': comment.context.code.substring(
-          param.right.start, param.right.end)
+  function paramWithDefaultToDoc(param) {
+    var newParam = paramToDoc(param.left);
+    var optionalParam = {
+      title: 'param',
+      name: newParam.name,
+      'default': comment.context.code.substring(
+        param.right.start, param.right.end)
+    };
+
+    if (newParam.type) {
+      optionalParam.type = {
+        type: 'OptionalType',
+        expression: newParam.type
       };
+    }
 
-      if (newParam.type) {
-        optionalParam.type = {
-          type: 'OptionalType',
-          expression: newParam.type
-        };
-      }
+    return optionalParam;
+  }
 
-      return optionalParam;
+  function paramToDoc(param) {
+    // ES6 default
+    if (param.type === 'AssignmentPattern') {
+      return paramWithDefaultToDoc(param);
     }
 
     var newParam = {

lib/output/markdown_ast.js

@@ -85,7 +85,7 @@ function commentsToAST(comments, opts, callback) {
       !!comment.members.instance.length &&
         u('root', comment.members.instance.map(generate.bind(null, depth + 1))),
       !!comment.members.static.length &&
-        u('root', comment.members.static.map(generate.bind(null, depth + 1))),
+        u('root', comment.members.static.map(generate.bind(null, depth + 1)))
     ].filter(identity));
   }
 

test/bin.js

@@ -58,7 +58,7 @@ test('accepts config file', function (t) {
     function (err, data) {
       t.error(err);
       if (process.env.UPDATE) {
-        var expected = fs.writeFileSync(
+        fs.writeFileSync(
           path.resolve(__dirname,
             'fixture',
             'sorting/output.json'), JSON.stringify(normalize(data), null, 2), 'utf8');

test/lib/git/mock_repo.js

@@ -16,5 +16,5 @@ module.exports = {
         'index.js': 'module.exports = 42;'
       }
     }
-  },
+  }
 };