chore: update devDeps and apply comma-dangle linting rule

Author: brettz9

package.json

@@ -21,19 +21,19 @@
     "@babel/plugin-transform-flow-strip-types": "^7.4.4",
     "@babel/preset-env": "^7.5.5",
     "@babel/register": "^7.5.5",
-    "@typescript-eslint/parser": "^1.12.0",
+    "@typescript-eslint/parser": "^1.13.0",
     "babel-eslint": "^10.0.2",
     "babel-plugin-add-module-exports": "^1.0.2",
-    "babel-plugin-istanbul": "^5.1.4",
+    "babel-plugin-istanbul": "^5.2.0",
     "chai": "^4.2.0",
-    "eslint": "^6.0.1",
-    "eslint-config-canonical": "^17.1.4",
+    "eslint": "^6.1.0",
+    "eslint-config-canonical": "^17.3.3",
     "gitdown": "^3.1.1",
     "glob": "^7.1.4",
-    "husky": "^3.0.1",
+    "husky": "^3.0.2",
     "mocha": "^6.2.0",
     "nyc": "^14.1.1",
-    "semantic-release": "^15.13.18",
+    "semantic-release": "^15.13.19",
     "typescript": "^3.5.3"
   },
   "engines": {

src/bin/generateReadme.js

@@ -60,7 +60,7 @@ const getAssertions = () => {
 
     return {
       invalid: _.map(codes.invalid, formatCodeSnippet),
-      valid: _.map(codes.valid, formatCodeSnippet)
+      valid: _.map(codes.valid, formatCodeSnippet),
     };
   });
 
@@ -81,8 +81,8 @@ const generateReadme = async () => {
   gitdown.setConfig({
     gitinfo: {
       defaultBranchName: getSomeBranch() || 'master',
-      gitPath: path.join(__dirname, '../../.git')
-    }
+      gitPath: path.join(__dirname, '../../.git'),
+    },
   });
   let documentBody = await gitdown.get();
 

src/exportParser.js

@@ -4,7 +4,7 @@ const debug = debugModule('requireExportJsdoc');
 
 const createNode = function () {
   return {
-    props: {}
+    props: {},
   };
 };
 
@@ -392,7 +392,7 @@ const parse = function (ast, node, opt) {
     ancestorsOnly: false,
     esm: true,
     initModuleExports: true,
-    initWindow: true
+    initWindow: true,
   };
 
   const globalVars = createNode();
@@ -414,7 +414,7 @@ const parse = function (ast, node, opt) {
   }
 
   return {
-    globalVars
+    globalVars,
   };
 };
 
@@ -424,5 +424,5 @@ const isExported = function (node, parseResult, opt) {
 
 export default {
   isExported,
-  parse
+  parse,
 };

src/index.js

@@ -56,9 +56,9 @@ export default {
         'jsdoc/require-returns-check': 'warn',
         'jsdoc/require-returns-description': 'warn',
         'jsdoc/require-returns-type': 'warn',
-        'jsdoc/valid-types': 'warn'
-      }
-    }
+        'jsdoc/valid-types': 'warn',
+      },
+    },
   },
   rules: {
     'check-alignment': checkAlignment,
@@ -86,6 +86,6 @@ export default {
     'require-returns-check': requireReturnsCheck,
     'require-returns-description': requireReturnsDescription,
     'require-returns-type': requireReturnsType,
-    'valid-types': validTypes
-  }
+    'valid-types': validTypes,
+  },
 };

src/iterateJsdoc.js

@@ -55,18 +55,18 @@ const parseComment = (commentNode, indent, trim = true) => {
           if (result) {
             return {
               data: {
-                description: result[1] === undefined ? '' : result[1]
+                description: result[1] === undefined ? '' : result[1],
               },
-              source: result[0]
+              source: result[0],
             };
           }
 
           // Always has at least whitespace due to `indent` we've added
           /* istanbul ignore next */
           return null;
-        }
+        },
     ],
-    trim
+    trim,
   })[0] || {};
 };
 
@@ -80,7 +80,7 @@ const getUtils = (
     implementsReplacesDocs,
     augmentsExtendsReplacesDocs,
     maxLines,
-    minLines
+    minLines,
   },
   report,
   context
@@ -142,7 +142,7 @@ const getUtils = (
       if (skipReportingBlockedTag) {
         return {
           blocked: true,
-          tagName
+          tagName,
         };
       }
       const message = isObject && ret.message || defaultMessage;
@@ -265,7 +265,7 @@ const getUtils = (
     const classNode = utils.getClassNode();
     const classJsdocNode = getJSDocComment(sourceCode, classNode, {
       maxLines,
-      minLines
+      minLines,
     });
 
     if (classJsdocNode) {
@@ -286,15 +286,15 @@ const getUtils = (
   utils.forEachPreferredTag = (tagName, arrayHandler, skipReportingBlockedTag = false) => {
     const targetTagName = utils.getPreferredTagName({
       skipReportingBlockedTag,
-      tagName
+      tagName,
     });
     if (!targetTagName ||
       skipReportingBlockedTag && targetTagName && typeof targetTagName === 'object'
     ) {
       return;
     }
     const matchingJsdocTags = _.filter(jsdoc.tags || [], {
-      tag: targetTagName
+      tag: targetTagName,
     });
 
     matchingJsdocTags.forEach((matchingJsdocTag) => {
@@ -307,10 +307,10 @@ const getUtils = (
       loc: {
         start: {
           column: 1,
-          line: 1
-        }
+          line: 1,
+        },
       },
-      message
+      message,
     });
   };
 
@@ -354,7 +354,7 @@ const makeReport = (context, commentNode) => {
 
       loc = {
         end: {line: lineNumber},
-        start: {line: lineNumber}
+        start: {line: lineNumber},
       };
       if (jsdocLoc.column) {
         const colNumber = commentNode.loc.start.column + jsdocLoc.column;
@@ -369,7 +369,7 @@ const makeReport = (context, commentNode) => {
       fix,
       loc,
       message,
-      node: commentNode
+      node: commentNode,
     });
   };
 
@@ -429,19 +429,19 @@ const iterateAllJsdocs = (iterator, ruleConfig) => {
               report,
               settings,
               sourceCode,
-              utils: getUtils(null, jsdoc, jsdocNode, settings, report, context)
+              utils: getUtils(null, jsdoc, jsdocNode, settings, report, context),
             });
           });
-        }
+        },
       };
     },
-    meta: ruleConfig.meta
+    meta: ruleConfig.meta,
   };
 };
 
 export {
   getSettings,
-  parseComment
+  parseComment,
 };
 
 /**
@@ -464,7 +464,7 @@ export default function iterateJsdoc (iterator, ruleConfig) {
   if (ruleConfig.iterateAllJsdocs) {
     return iterateAllJsdocs(iterator, {
       meta: ruleConfig.meta,
-      noTrim: ruleConfig.noTrim
+      noTrim: ruleConfig.noTrim,
     });
   }
 
@@ -521,7 +521,7 @@ export default function iterateJsdoc (iterator, ruleConfig) {
           report,
           settings,
           sourceCode,
-          utils
+          utils,
         });
       };
 
@@ -534,9 +534,9 @@ export default function iterateJsdoc (iterator, ruleConfig) {
       return {
         ArrowFunctionExpression: checkJsdoc,
         FunctionDeclaration: checkJsdoc,
-        FunctionExpression: checkJsdoc
+        FunctionExpression: checkJsdoc,
       };
     },
-    meta: ruleConfig.meta
+    meta: ruleConfig.meta,
   };
 }

src/jsdocUtils.js

@@ -40,7 +40,7 @@ const getJsdocParameterNamesDeep = (jsdoc : Object, targetTagName : string) : Ar
   let jsdocParameterNames;
 
   jsdocParameterNames = _.filter(jsdoc.tags, {
-    tag: targetTagName
+    tag: targetTagName,
   });
 
   jsdocParameterNames = _.map(jsdocParameterNames, 'name');
@@ -144,7 +144,7 @@ const namepathDefiningTags = [
   'interface',
   'member', 'var',
   'mixin',
-  'namespace'
+  'namespace',
 ];
 
 const namepathPointingTags = [
@@ -163,7 +163,7 @@ const namepathPointingTags = [
   // MAY BE USEFUL WITHOUT NAMEPATH
   'emits',
   'fires',
-  'listens'
+  'listens',
 ];
 
 const isNamepathDefiningTag = (tagName) => {
@@ -219,11 +219,11 @@ let tagsWithTypes = [
   'throws',
   'type',
   'typedef',
-  'yields'
+  'yields',
 ];
 
 const closureTagsWithTypes = [
-  'package', 'private', 'protected', 'public', 'static'
+  'package', 'private', 'protected', 'public', 'static',
 ];
 
 const tagsWithTypesAliases = [
@@ -235,7 +235,7 @@ const tagsWithTypesAliases = [
   'prop',
   'return',
   'exception',
-  'yield'
+  'yield',
 ];
 
 tagsWithTypes = tagsWithTypes.concat(tagsWithTypesAliases, closureTagsWithTypes);
@@ -366,8 +366,8 @@ const enforcedContexts = (context, defaultContexts) => {
     contexts = defaultContexts === true ? [
       'ArrowFunctionExpression',
       'FunctionDeclaration',
-      'FunctionExpression'
-    ] : defaultContexts
+      'FunctionExpression',
+    ] : defaultContexts,
   } = context.options[0] || {};
 
   return contexts;
@@ -393,7 +393,7 @@ const tagsWithNamesAndDescriptions = [
   'param', 'arg', 'argument', 'property', 'prop',
 
   // These two are parsed by our custom parser as though having a `name`
-  'returns', 'return'
+  'returns', 'return',
 ];
 
 const getTagsByType = (tags, tagPreference) => {
@@ -411,7 +411,7 @@ const getTagsByType = (tags, tagPreference) => {
 
   return {
     tagsWithNames,
-    tagsWithoutNames
+    tagsWithoutNames,
   };
 };
 
@@ -457,5 +457,5 @@ export default {
   isTagWithMandatoryNamepathOrType,
   isTagWithType,
   isValidTag,
-  parseClosureTemplateTag
+  parseClosureTemplateTag,
 };

src/rules/checkAlignment.js

@@ -5,7 +5,7 @@ export default iterateJsdoc(({
   sourceCode,
   jsdocNode,
   report,
-  indent
+  indent,
 }) => {
   // `indent` is whitespace from line 1 (`/**`), so slice and account for "/".
   const indentLevel = indent.length + 1;
@@ -34,7 +34,7 @@ export default iterateJsdoc(({
   sourceLines.some((line, lineNum) => {
     if (line.length !== indentLevel) {
       report('Expected JSDoc block to be aligned.', fix, {
-        line: lineNum + 1
+        line: lineNum + 1,
       });
 
       return true;
@@ -46,6 +46,6 @@ export default iterateJsdoc(({
   iterateAllJsdocs: true,
   meta: {
     fixable: 'code',
-    type: 'layout'
-  }
+    type: 'layout',
+  },
 });