feat: support ESLint 8.x #32
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MichaelDeBoey
commented
Aug 23, 2021
| Legacy: { ConfigValidator }, | ||
| } = require("@eslint/eslintrc") | ||
| const { rules: PluginRulesIndex } = require("@mysticatea/eslint-plugin") | ||
| const { rules: removedRules } = require("eslint/conf/replacements.json") |
There was a problem hiding this comment.
@mysticatea We need to figure out a way to get this file, as it's not exported anymore since ESLint v8
This is currently giving the following error when running tests
> @mysticatea/[email protected] test
> npm run -s lint && nyc mocha "tests/lib/**/*.js" --reporter dot
.../eslint-plugin/node_modules/yargs/yargs.js:1172
else throw err
^
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './conf/replacements.json' is not defined by "exports" in .../eslint-plugin/node_modules/eslint/package.json
at new NodeError (node:internal/errors:370:5)
at throwExportsNotFound (node:internal/modules/esm/resolve:335:9)
at packageExportsResolve (node:internal/modules/esm/resolve:560:3)
at resolveExports (node:internal/modules/cjs/loader:476:36)
at Function.Module._findPath (node:internal/modules/cjs/loader:516:31)
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:913:27)
at Function.Module._load (node:internal/modules/cjs/loader:772:27)
at Module.require (node:internal/modules/cjs/loader:999:19)
at require (node:internal/modules/cjs/helpers:93:18)
at Object.<anonymous> (.../eslint-plugin/tests/lib/configs/_rules.js:13:33)
at Module._compile (node:internal/modules/cjs/loader:1095:14)
at Module.replacementCompile (.../eslint-plugin/node_modules/append-transform/index.js:58:13)
at Module._extensions..js (node:internal/modules/cjs/loader:1124:10)
at Object.<anonymous> (.../eslint-plugin/node_modules/append-transform/index.js:62:4)
at Module.load (node:internal/modules/cjs/loader:975:32)
at Function.Module._load (node:internal/modules/cjs/loader:816:12)
at Module.require (node:internal/modules/cjs/loader:999:19)
at require (node:internal/modules/cjs/helpers:93:18)
at .../eslint-plugin/node_modules/mocha/lib/mocha.js:334:36
at Array.forEach (<anonymous>)
at Mocha.loadFiles (.../eslint-plugin/node_modules/mocha/lib/mocha.js:331:14)
at Mocha.run (.../eslint-plugin/node_modules/mocha/lib/mocha.js:809:10)
at Object.exports.singleRun .../eslint-plugin/node_modules/mocha/lib/cli/run-helpers.js:108:16)
at exports.runMocha .../eslint-plugin/node_modules/mocha/lib/cli/run-helpers.js:142:13)
at Object.exports.handler (.../eslint-plugin/node_modules/mocha/lib/cli/run.js:292:3)
at Object.runCommand (.../eslint-plugin/node_modules/yargs/lib/command.js:242:26)
at Object.parseArgs [as _parseArgs] (.../eslint-plugin/node_modules/yargs/yargs.js:1096:28)
at Object.parse (.../eslint-plugin/node_modules/yargs/yargs.js:575:25)
at Object.exports.main (.../eslint-plugin/node_modules/mocha/lib/cli/cli.js:68:6)
at Object.<anonymous> (.../eslint-plugin/node_modules/mocha/bin/mocha:162:29)
at Module._compile (node:internal/modules/cjs/loader:1095:14)
at Module.replacementCompile (.../eslint-plugin/node_modules/append-transform/index.js:58:13)
at Module._extensions..js (node:internal/modules/cjs/loader:1124:10)
at Object.<anonymous> (.../eslint-plugin/node_modules/append-transform/index.js:62:4)
at Module.load (node:internal/modules/cjs/loader:975:32)
at Function.Module._load (node:internal/modules/cjs/loader:816:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)
at runMain (.../.node-spawn-wrap-74801-8936ba6826e7/node:68:10)
at Function.<anonymous> (.../.node-spawn-wrap-74801-8936ba6826e7/node:171:5)
at Object.<anonymous> (.../eslint-plugin/node_modules/nyc/bin/wrap.js:27:4)
at Module._compile (node:internal/modules/cjs/loader:1095:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1124:10)
at Module.load (node:internal/modules/cjs/loader:975:32)
at Function.Module._load (node:internal/modules/cjs/loader:816:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)
at .../.node-spawn-wrap-74801-8936ba6826e7/node:178:8
at Object.<anonymous> (.../.node-spawn-wrap-74801-8936ba6826e7/node:181:3)
at Module._compile (node:internal/modules/cjs/loader:1095:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1124:10)
at Module.load (node:internal/modules/cjs/loader:975:32)
at Function.Module._load (node:internal/modules/cjs/loader:816:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12)
at node:internal/main/run_main_module:17:47 {
code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}
=============================== Coverage summary ===============================
Statements : Unknown% ( 0/0 )
Branches : Unknown% ( 0/0 )
Functions : Unknown% ( 0/0 )
Lines : Unknown% ( 0/0 )
================================================================================
This was referenced Aug 23, 2021
16 tasks
MichaelDeBoey
force-pushed
the
eslint-8
branch
from
776fb8a to
c51cd08
Compare
BREAKING CHANGE: Requires Node@^10.12.0 || >=12.0.0 BREAKING CHANGE: Requires ESLint@^7.0.0
MichaelDeBoey
force-pushed
the
eslint-8
branch
from
c51cd08 to
5b30198
Compare
4 tasks
MichaelDeBoey
force-pushed
the
eslint-8
branch
2 times, most recently
from
0b7b979 to
db13d9e
Compare
BREAKING CHANGE: Requires Node@^12.22.0 || ^14.17.0 || >=16.0.0 BREAKING CHANGE: Requires ESLint@^8.0.0
MichaelDeBoey
force-pushed
the
eslint-8
branch
from
db13d9e to
43cd406
Compare
|
|
|
|
MichaelDeBoey
force-pushed
the
eslint-8
branch
2 times, most recently
from
90f2b3e to
43cd406
Compare
|
For people watching this PR: we've already started with our own fork in order to not hold the wider community back anymore: https://github.com/eslint-community/eslint-plugin-mysticatea @mysticatea We would still love to move the original repo to the new https://github.com/eslint-community though. This PR is released in |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ESLint v8.0.0 is released 🎉
Dependencies should be compatible with ESLint 8 too before we can merge this one:
@typescript-eslint/eslint-plugin(Support for ESLint v8 typescript-eslint/typescript-eslint#3738)meta.docs.categoryfrom rules typescript-eslint/typescript-eslint#3800PrivateIdentifiertypescript-eslint/typescript-eslint#3808v5.0.0@typescript-eslint/parser(Support for ESLint v8 typescript-eslint/typescript-eslint#3738)meta.docs.categoryfrom rules typescript-eslint/typescript-eslint#3800PrivateIdentifiertypescript-eslint/typescript-eslint#3808v5.0.0eslint-plugin-eslint-comments(Support ESLint 8.x eslint-plugin-eslint-comments#62)eslint-plugin-eslint-plugin(Support ESLint 8.x eslint-community/eslint-plugin-eslint-plugin#175)meta.hasSuggestionsproperty tometa-property-orderingrule eslint-community/eslint-plugin-eslint-plugin#119v4.0.0-0v4.0.0eslint-plugin-node(Support ESLint 8.x eslint-plugin-node#294)eslint-plugin-prettier(Support ESLint 8.x prettier/eslint-plugin-prettier#427)Releaseeslint-plugin-vue(Support ESLint 8.x vuejs/eslint-plugin-vue#1615)v8.0.0BREAKING CHANGE: Requires Node@^12.22.0 || ^14.17.0 || >=16.0.0
BREAKING CHANGE: Requires ESLint@^8.0.0
Closes #31
This branch is dependent on #29