-
-
Notifications
You must be signed in to change notification settings - Fork 592
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix(eslint): Prepare eslint for rollup 3 and upgrade eslint to newest…
… version (#1309)
- Loading branch information
1 parent
69146cd
commit 552728b
Showing
10 changed files
with
2,199 additions
and
1,989 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,10 +13,15 @@ | |
| "author": "Bogdan Chadkin <[email protected]>", | ||
| "homepage": "https://github.com/rollup/plugins/tree/master/packages/eslint#readme", | ||
| "bugs": "https://github.com/rollup/plugins/issues", | ||
| "main": "dist/index.js", | ||
| "module": "dist/index.es.js", | ||
| "main": "./dist/cjs/index.js", | ||
| "module": "./dist/es/index.js", | ||
| "exports": { | ||
| "import": "./dist/es/index.js", | ||
| "types": "./types/index.d.ts", | ||
| "default": "./dist/cjs/index.js" | ||
| }, | ||
| "engines": { | ||
| "node": ">= 10.0.0" | ||
| "node": ">=14.0.0" | ||
| }, | ||
| "scripts": { | ||
| "build": "rollup -c", | ||
|
|
@@ -34,6 +39,7 @@ | |
| }, | ||
| "files": [ | ||
| "dist", | ||
| "!dist/**/*.map", | ||
| "types", | ||
| "README.md", | ||
| "LICENSE" | ||
|
|
@@ -48,30 +54,26 @@ | |
| "lint" | ||
| ], | ||
| "peerDependencies": { | ||
| "rollup": "^1.20.0||^2.0.0" | ||
| "rollup": "^1.20.0||^2.0.0||^3.0.0" | ||
| }, | ||
| "peerDependenciesMeta": { | ||
| "rollup": { | ||
| "optional": true | ||
| } | ||
| }, | ||
| "dependencies": { | ||
| "@rollup/pluginutils": "^4.0.0", | ||
| "eslint": "^7.12.0" | ||
| "@rollup/pluginutils": "^4.2.1", | ||
| "eslint": "^8.24.0" | ||
| }, | ||
| "devDependencies": { | ||
| "@rollup/plugin-node-resolve": "^9.0.0", | ||
| "@rollup/plugin-typescript": "^6.0.0", | ||
| "@types/eslint": "^7.2.2", | ||
| "rollup": "^2.67.3", | ||
| "typescript": "^4.1.2" | ||
| "@rollup/plugin-node-resolve": "^14.1.0", | ||
| "@rollup/plugin-typescript": "^8.5.0", | ||
| "@types/eslint": "^8.4.6", | ||
| "rollup": "^3.0.0-7", | ||
| "typescript": "^4.8.3" | ||
| }, | ||
| "types": "types/index.d.ts", | ||
| "types": "./types/index.d.ts", | ||
| "ava": { | ||
| "babel": { | ||
| "compileEnhancements": false | ||
| }, | ||
| "extensions": [ | ||
| "ts" | ||
| ], | ||
| "require": [ | ||
| "ts-node/register" | ||
| ], | ||
| "files": [ | ||
| "!**/fixtures/**", | ||
| "!**/helpers/**", | ||
|
|
||
This file was deleted.
Oops, something went wrong.
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| import { readFileSync } from 'fs'; | ||
|
|
||
| import { createConfig } from '../../shared/rollup.config.mjs'; | ||
|
|
||
| export default createConfig({ | ||
| pkg: JSON.parse(readFileSync(new URL('./package.json', import.meta.url), 'utf8')) | ||
| }); |
This file contains 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,81 +1,83 @@ | ||
| import * as path from 'path'; | ||
| import { relative, resolve, sep } from 'path'; | ||
|
|
||
| import { Plugin } from 'rollup'; | ||
| import { createFilter } from '@rollup/pluginutils'; | ||
| import { CLIEngine } from 'eslint'; | ||
| import { ESLint } from 'eslint'; | ||
|
|
||
| import { RollupEslintOptions } from '../types'; | ||
| import type { RollupEslintOptions } from '../types'; | ||
|
|
||
| function normalizePath(id: string) { | ||
| return path.relative(process.cwd(), id).split(path.sep).join('/'); | ||
| return relative(process.cwd(), id).split(sep).join('/'); | ||
| } | ||
|
|
||
| export default function eslint(options = {} as RollupEslintOptions): Plugin { | ||
| if (typeof options === 'string') { | ||
| const configFile = path.resolve(process.cwd(), options); | ||
| const configFile = resolve(process.cwd(), options); | ||
| // eslint-disable-next-line global-require, import/no-dynamic-require, no-param-reassign | ||
| options = require(configFile); | ||
| // Tell eslint not to look for configuration files. | ||
| // eslint-disable-next-line no-param-reassign | ||
| options.useEslintrc = false; | ||
| } | ||
|
|
||
| const cli = new CLIEngine(options); | ||
| let formatter: CLIEngine.Formatter; | ||
|
|
||
| switch (typeof options.formatter) { | ||
| case 'string': | ||
| formatter = cli.getFormatter(options.formatter); | ||
| break; | ||
| case 'function': | ||
| ({ formatter } = options); | ||
| break; | ||
| default: | ||
| formatter = cli.getFormatter('stylish'); | ||
| } | ||
| const { | ||
| include, | ||
| exclude = /node_modules/, | ||
| throwOnWarning = false, | ||
| throwOnError = false, | ||
| formatter = 'stylish', | ||
| ...eslintOptions | ||
| } = options; | ||
|
|
||
| const filter = createFilter(options.include, options.exclude || /node_modules/); | ||
| const eslintInstance = new ESLint(eslintOptions); | ||
| const filter = createFilter(include, exclude); | ||
|
|
||
| return { | ||
| name: 'eslint', | ||
|
|
||
| // eslint-disable-next-line consistent-return | ||
| transform(code, id) { | ||
| async transform(_, id: string) { | ||
| const file = normalizePath(id); | ||
| if (!filter(id) || cli.isPathIgnored(file)) { | ||
| if (!filter(id) || (await eslintInstance.isPathIgnored(file))) { | ||
| return null; | ||
| } | ||
|
|
||
| const report = cli.executeOnText(code, file); | ||
| const hasWarnings = options.throwOnWarning && report.warningCount !== 0; | ||
| const hasErrors = options.throwOnError && report.errorCount !== 0; | ||
| const results = await eslintInstance.lintFiles(file); | ||
| const [result] = results; | ||
|
|
||
| if (options.fix && report) { | ||
| CLIEngine.outputFixes(report); | ||
| if (eslintOptions.fix) { | ||
| await ESLint.outputFixes(results); | ||
| } | ||
|
|
||
| if (report.warningCount === 0 && report.errorCount === 0) { | ||
| if (result.warningCount === 0 && result.errorCount === 0) { | ||
| return null; | ||
| } | ||
|
|
||
| const result = formatter(report.results); | ||
| const eslintFormatter: ESLint.Formatter = | ||
| typeof formatter === 'string' | ||
| ? await eslintInstance.loadFormatter(formatter) | ||
| : { format: formatter }; | ||
| const output = eslintFormatter.format(results); | ||
|
|
||
| if (result) { | ||
| if (output) { | ||
| // eslint-disable-next-line no-console | ||
| console.log(result); | ||
| console.log(output); | ||
| } | ||
|
|
||
| if (hasWarnings && hasErrors) { | ||
| throw Error('Warnings or errors were found'); | ||
| const errorMessages = []; | ||
| if (result.warningCount > 0 && throwOnWarning) { | ||
| errorMessages.push(`${result.warningCount} warning${result.warningCount > 1 ? 's' : ''}`); | ||
| } | ||
|
|
||
| if (hasWarnings) { | ||
| throw Error('Warnings were found'); | ||
| if (result.errorCount > 0 && throwOnError) { | ||
| errorMessages.push(`${result.errorCount} error${result.errorCount > 1 ? 's' : ''}`); | ||
| } | ||
|
|
||
| if (hasErrors) { | ||
| throw Error('Errors were found'); | ||
| if (errorMessages.length > 0) { | ||
| throw new Error( | ||
| `Found ${errorMessages.join(' and ')} in ${relative('.', result.filePath)}` | ||
| ); | ||
| } | ||
|
|
||
| return null; | ||
| } | ||
| }; | ||
| } |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Oops, something went wrong.