GeezerVison is showing github.com — tools: enable linter on some fixtures file · nodejs/node@98df744
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: eslint.config.mjs +17-13Lines changed: 17 additions & 13 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -14,6 +14,7 @@ import { | 14 | 14 | } from './tools/eslint/eslint.config_utils.mjs'; | 15 | 15 | import nodeCore from './tools/eslint/eslint-plugin-node-core.js'; | 16 | 16 |
| | 17 | +const { globalIgnores } = await importEslintTool('eslint/config'); | 17 | 18 | const { default: js } = await importEslintTool('@eslint/js'); | 18 | 19 | const { default: babelEslintParser } = await importEslintTool('@babel/eslint-parser'); | 19 | 20 | const babelPluginSyntaxImportAttributes = resolveEslintTool('@babel/plugin-syntax-import-attributes'); | @@ -39,19 +40,22 @@ Module._resolveFilename = (request, parent, isMain, options) => { | 39 | 40 |
| 40 | 41 | export default [ | 41 | 42 | // #region ignores | 42 | | - { | 43 | | - ignores: [ | 44 | | - '**/node_modules/**', | 45 | | - 'benchmark/fixtures/**', | 46 | | - 'benchmark/tmp/**', | 47 | | - 'doc/changelogs/CHANGELOG_V1*.md', | 48 | | - '!doc/changelogs/CHANGELOG_V18.md', | 49 | | - 'lib/punycode.js', | 50 | | - 'test/.tmp.*/**', | 51 | | - 'test/addons/??_*', | 52 | | - 'test/fixtures/**', | 53 | | - ], | 54 | | - }, | | 43 | + globalIgnores([ | | 44 | + '**/node_modules/**', | | 45 | + 'benchmark/fixtures/**', | | 46 | + 'benchmark/tmp/**', | | 47 | + 'doc/changelogs/CHANGELOG_V1*.md', | | 48 | + '!doc/changelogs/CHANGELOG_V18.md', | | 49 | + 'lib/punycode.js', | | 50 | + 'test/.tmp.*/**', | | 51 | + 'test/addons/??_*', | | 52 | + | | 53 | + // We want to lint only a few specific fixtures folders | | 54 | + 'test/fixtures/*', | | 55 | + '!test/fixtures/console', | | 56 | + '!test/fixtures/v8', | | 57 | + '!test/fixtures/vm', | | 58 | + ]), | 55 | 59 | // #endregion | 56 | 60 | // #region general config | 57 | 61 | js.configs.recommended, | |
Collapse file Copy file name to clipboardExpand all lines: test/eslint.config_partial.mjs +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -139,7 +139,7 @@ export default [ | 139 | 139 | }, | 140 | 140 | { | 141 | 141 | files: [ | 142 | | - 'test/{common,wpt}/**/*.{js,mjs,cjs}', | | 142 | + 'test/{common,fixtures,wpt}/**/*.{js,mjs,cjs}', | 143 | 143 | 'test/eslint.config_partial.mjs', | 144 | 144 | ], | 145 | 145 | rules: { | |
|