Skip to content

Commit 98df744

Browse files
aduh95RafaelGSS
authored andcommitted
tools: enable linter on some fixtures file
PR-URL: #57674 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>
1 parent 913a283 commit 98df744

File tree

2 files changed

+18
-14
lines changed

2 files changed

+18
-14
lines changed

eslint.config.mjs

Lines changed: 17 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ import {
1414
} from './tools/eslint/eslint.config_utils.mjs';
1515
import nodeCore from './tools/eslint/eslint-plugin-node-core.js';
1616

17+
const { globalIgnores } = await importEslintTool('eslint/config');
1718
const { default: js } = await importEslintTool('@eslint/js');
1819
const { default: babelEslintParser } = await importEslintTool('@babel/eslint-parser');
1920
const babelPluginSyntaxImportAttributes = resolveEslintTool('@babel/plugin-syntax-import-attributes');
@@ -39,19 +40,22 @@ Module._resolveFilename = (request, parent, isMain, options) => {
3940

4041
export default [
4142
// #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+
]),
5559
// #endregion
5660
// #region general config
5761
js.configs.recommended,

test/eslint.config_partial.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ export default [
139139
},
140140
{
141141
files: [
142-
'test/{common,wpt}/**/*.{js,mjs,cjs}',
142+
'test/{common,fixtures,wpt}/**/*.{js,mjs,cjs}',
143143
'test/eslint.config_partial.mjs',
144144
],
145145
rules: {

0 commit comments

Comments
 (0)