style(lint): resolve errors

1. Revert to sonar.branch instead of sonar.branch.name
2. Fix max-len warnings
3. Remove dependencies with nested vulnerabilities

#44,#187

.travis.yml

@@ -8,7 +8,7 @@ addons:
       secure:       "$GH_TOKEN"
     branches:
       - master
-      - /[build|chore|ci|docs|feat|fix|perf|refactor|revert|spike|style|test].*/
+      - /[build|chore|ci|docs|feat|fix|perf|refactor|revert|spike|style|test]\/.*/
 jdk:
   - oraclejdk8
 
@@ -37,7 +37,6 @@ branches:
 
 before_script:
   - npm prune
-  - npm dedupe
 
 script:
   - echo "$TRAVIS_PULL_REQUEST_BRANCH"
@@ -47,7 +46,7 @@ after_script:
   - cat docs/reports/coverage/lcov.info | npx coveralls
   # Send stats to SonarCloud for pull requests
   - test $TRAVIS_PULL_REQUEST != false
-    && sonar-scanner -Dproject.settings=./sonar-project.properties -Dsonar.branch.name="$TRAVIS_PULL_REQUEST_BRANCH" -Dsonar.branch.target=master -Dsonar.organization=$SONAR_ORGANIZATION -Dsonar.login=$SONAR_TOKEN -X
+    && sonar-scanner -Dproject.settings=./sonar-project.properties -Dsonar.branch=$TRAVIS_PULL_REQUEST_BRANCH -Dsonar.organization=$SONAR_ORGANIZATION -Dsonar.login=$SONAR_TOKEN -X
 
 jobs:
   include:

lib/crc/crc-responsibility.js

@@ -59,8 +59,6 @@ const getPrimaryDescription =
 class CrcResponsibility {
   constructor (crcClass) {
     this.comments = getComments(crcClass);
-
-    // This.all = getDescriptionsFromTags(this.comments);
   }
 
   /**
@@ -90,11 +88,12 @@ class CrcResponsibility {
 
   static descriptionFromContext (context) {
     const cxtComments = get(context, "code.ast.comments", []);
-    const comments = cxtComments.map((comment) => doctrine.parse(comment.value, {
-      "recoverable": true,
-      "sloppy": true,
-      "unwrap": true
-    }));
+    const comments =
+      cxtComments.map((comment) => doctrine.parse(comment.value, {
+        "recoverable": true,
+        "sloppy": true,
+        "unwrap": true
+      }));
 
     const description =
       getPrimaryDescription(comments) || getDescriptionFromTags(comments);

lib/crc/prototypable.js

@@ -53,7 +53,8 @@ const visitClass = (context) => {
  */
 
 const visitNewExpression = (context) => {
-  const descriptor = "ExpressionStatement > [left.property.name=\"prototype\"]" +
+  const descriptor =
+    "ExpressionStatement > [left.property.name=\"prototype\"]" +
     "[right.type=\"NewExpression\"]";
   const node = queryContext(context, descriptor);
   return {

lib/crc/source-code-factory.js

@@ -11,7 +11,8 @@ const {SourceCode} = require("eslint");
  */
 
 // eslint-disable-next-line security/detect-non-literal-fs-filename
-const toSourceString = (filePath) => fs.readFileSync(path.resolve(filePath)).toString();
+const toSourceString =
+  (filePath) => fs.readFileSync(path.resolve(filePath)).toString();
 
 /**
  * A helper object that generates ESLint

lib/formatters/md/index.js

@@ -127,24 +127,18 @@ module.exports = function format (results) {
   const crcReporter = new CrcReporter();
   const crcModels = crcReporter.report(results);
 
-  // Const codebase = crcReporter.codebase;
-
   const toc = createTableOfContents(crcModels, tocFactory);
 
-  // Console.log(codebase);
-
   decorateBuiltInsWithLinks(crcModels);
   const models = crcModels.map((crcModel) => crcModelTemplate({
     crcModel,
     tocFactory
   })).join("");
 
-  const report = crcPageTemplate({
+  return crcPageTemplate({
     "date": new Date(),
     "results": models,
     "summary": `Object count: ${crcModels.length}`,
     toc
   });
-
-  return report;
 };

lib/formatters/md/toc-factory.js

@@ -3,7 +3,8 @@ const markdownToc = require("markdown-toc");
 
 const mdnRef = new MdnReference();
 
-const toSlug = (crcModel) => `${crcModel.class.meta.kind} ${crcModel.class.name}`;
+const toSlug =
+  (crcModel) => `${crcModel.class.meta.kind} ${crcModel.class.name}`;
 
 class TocFactory {
   static toAnchor (crcModel) {