chore: restore Node versions; fixes gajus#801

Also:
1. Adds `engine-strict` flag to identify problematic dev. projects for testing on older Node.
2. Adds legacy-peer-deps for older combos of overwriting no-saves
3. Sets `eslint-plugin-mocha` to older version (not supporting Node 12)
4. Changes tests slightly to adjust for different `eslint-config-canonical` behavior

Author: brettz9

.travis.yml

@@ -2,28 +2,38 @@ os: linux
 dist: xenial
 language: node_js
 node_js:
+  - "12.22.0"
+  - "14.17.0"
   - "16"
-  # - "14.14.0"
-  # - "12.20.0"
 install:
-  - echo "Avoid Travis's npm auto-install"
+  - npm install --legacy-peer-deps
 before_script: >
+
+  node_version=$(node -v);
+
   if [ ${ESLINT} = "7" ]; then
-    npm install --no-save "eslint@${ESLINT}" [email protected]
+    if [ ${node_version:1:2} = "16" ]; then
+      npm install --legacy-peer-deps --engine-strict --no-save "eslint@${ESLINT}" [email protected] eslint-plugin-mocha@9
+    else
+      npm install --legacy-peer-deps --no-save "eslint@${ESLINT}" [email protected] eslint-plugin-mocha@9
+    fi
   else
-    npm install --no-save "eslint@${ESLINT}"
-    npm run lint
+    if [ ${node_version:1:2} = "16" ]; then
+      npm install --engine-strict --no-save "eslint@${ESLINT}"
+      npm run lint
+    else
+      npm install --legacy-peer-deps --no-save "eslint@${ESLINT}" [email protected] eslint-plugin-mocha@9
+    fi
   fi
 
 notifications:
   email: false
 script:
   - npm run test
-  - npm run build
 env:
   jobs:
-    - ESLINT=8
     - ESLINT=7
+    - ESLINT=8
 after_success:
   - export NODE_ENV=production
   - npm run build

README.md

@@ -1259,33 +1259,33 @@ function quux () {}
 // Message: @example error (semi): Missing semicolon.
 
 /**
- * @example const i = 5;
- * quux2()
+ * @example const j = 5;
+ * quux2();
  */
 function quux2 () {
 
 }
 // "jsdoc/check-examples": ["error"|"warn", {"matchingFileName":"../../jsdocUtils.js"}]
-// Message: @example warning (id-length): Identifier name 'i' is too short (< 2).
+// Message: @example warning (id-length): Identifier name 'j' is too short (< 2).
 
 /**
- * @example const i = 5;
- * quux2()
+ * @example const k = 5;
+ * quux2();
  */
 function quux2 () {
 
 }
 // "jsdoc/check-examples": ["error"|"warn", {"configFile":".eslintrc.json","matchingFileName":"../../jsdocUtils.js"}]
-// Message: @example warning (id-length): Identifier name 'i' is too short (< 2).
+// Message: @example warning (id-length): Identifier name 'k' is too short (< 2).
 
 /**
- * @example const i = 5;
- * quux2()
+ * @example const m = 5;
+ * quux2();
  */
 function quux2 () {
 
 }
-// Message: @example warning (id-length): Identifier name 'i' is too short (< 2).
+// Message: @example warning (id-length): Identifier name 'm' is too short (< 2).
 
 /**
  * @example const i = 5;

test/rules/assertions/checkExamples.js

@@ -364,8 +364,8 @@ export default {
     {
       code: `
           /**
-           * @example const i = 5;
-           * quux2()
+           * @example const j = 5;
+           * quux2();
            */
           function quux2 () {
 
@@ -374,11 +374,7 @@ export default {
       errors: [
         {
           line: 3,
-          message: '@example warning (id-length): Identifier name \'i\' is too short (< 2).',
-        },
-        {
-          line: 4,
-          message: '@example error (semi): Missing semicolon.',
+          message: '@example warning (id-length): Identifier name \'j\' is too short (< 2).',
         },
       ],
       options: [
@@ -390,8 +386,8 @@ export default {
     {
       code: `
           /**
-           * @example const i = 5;
-           * quux2()
+           * @example const k = 5;
+           * quux2();
            */
           function quux2 () {
 
@@ -400,11 +396,7 @@ export default {
       errors: [
         {
           line: 3,
-          message: '@example warning (id-length): Identifier name \'i\' is too short (< 2).',
-        },
-        {
-          line: 4,
-          message: '@example error (semi): Missing semicolon.',
+          message: '@example warning (id-length): Identifier name \'k\' is too short (< 2).',
         },
       ],
       options: [
@@ -417,8 +409,8 @@ export default {
     {
       code: `
           /**
-           * @example const i = 5;
-           * quux2()
+           * @example const m = 5;
+           * quux2();
            */
           function quux2 () {
 
@@ -427,11 +419,7 @@ export default {
       errors: [
         {
           line: 3,
-          message: '@example warning (id-length): Identifier name \'i\' is too short (< 2).',
-        },
-        {
-          line: 4,
-          message: '@example error (semi): Missing semicolon.',
+          message: '@example warning (id-length): Identifier name \'m\' is too short (< 2).',
         },
       ],
       filename: 'test/rules/data/jsdocUtils.js',