Upgrade: @eslint/eslintrc@1.0.3 for Jest workaround (#15164)

btmills · web-flow · commit f9217e527e1c · 2021-10-13T22:45:55.000-04:00
* Upgrade: @eslint/eslintrc@1.0.3 for Jest workaround

* Chore: Remove unused disable comments

`eslint-plugin-node`'s `no-missing-require` previously had a false
positive report that the `@eslint/eslintrc/universal` export did not
exist. Now with the release of `@eslint/eslintrc@1.0.3` with a "fake"
(not part of `exports` in `package.json`) `universal` export, as a
workaround for Jest, `no-missing-require` is satisfied that it does
exist.
diff --git a/lib/linter/config-comment-parser.js b/lib/linter/config-comment-parser.js
@@ -15,7 +15,7 @@ const levn = require("levn"),
         Legacy: {
             ConfigOps
         }
-    } = require("@eslint/eslintrc/universal"); // eslint-disable-line node/no-missing-require -- false positive
+    } = require("@eslint/eslintrc/universal");
 
 const debug = require("debug")("eslint:config-comment-parser");
 
diff --git a/lib/linter/linter.js b/lib/linter/linter.js
@@ -24,7 +24,7 @@ const
             ConfigValidator,
             environments: BuiltInEnvironments
         }
-    } = require("@eslint/eslintrc/universal"), // eslint-disable-line node/no-missing-require -- false positive
+    } = require("@eslint/eslintrc/universal"),
     Traverser = require("../shared/traverser"),
     { SourceCode } = require("../source-code"),
     CodePathAnalyzer = require("./code-path-analysis/code-path-analyzer"),
diff --git a/package.json b/package.json
@@ -47,7 +47,7 @@
   "homepage": "https://eslint.org",
   "bugs": "https://github.com/eslint/eslint/issues/",
   "dependencies": {
-    "@eslint/eslintrc": "^1.0.2",
+    "@eslint/eslintrc": "^1.0.3",
     "@humanwhocodes/config-array": "^0.6.0",
     "ajv": "^6.10.0",
     "chalk": "^4.0.0",

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ const levn = require("levn"),`
`15`	`15`	`Legacy: {`
`16`	`16`	`ConfigOps`
`17`	`17`	`}`
`18`		`- } = require("@eslint/eslintrc/universal"); // eslint-disable-line node/no-missing-require -- false positive`
	`18`	`+ } = require("@eslint/eslintrc/universal");`
`19`	`19`
`20`	`20`	`const debug = require("debug")("eslint:config-comment-parser");`
`21`	`21`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ const`
`24`	`24`	`ConfigValidator,`
`25`	`25`	`environments: BuiltInEnvironments`
`26`	`26`	`}`
`27`		`- } = require("@eslint/eslintrc/universal"), // eslint-disable-line node/no-missing-require -- false positive`
	`27`	`+ } = require("@eslint/eslintrc/universal"),`
`28`	`28`	`Traverser = require("../shared/traverser"),`
`29`	`29`	`{ SourceCode } = require("../source-code"),`
`30`	`30`	`CodePathAnalyzer = require("./code-path-analysis/code-path-analyzer"),`