Switch back to use eslintignore file for sdl tooling

However use a different filename to avoid a warning

.eslint-ignore

@@ -0,0 +1,36 @@
+**/build/*/**/*.js
+**/dist/**/*.js
+**/extensions/**/*.d.ts
+**/extensions/**/build/**
+**/extensions/**/colorize-fixtures/**
+**/extensions/css-language-features/server/test/pathCompletionFixtures/**
+**/extensions/html-language-features/server/lib/jquery.d.ts
+**/extensions/html-language-features/server/src/test/pathCompletionFixtures/**
+**/extensions/ipynb/notebook-out/**
+**/extensions/markdown-language-features/media/**
+**/extensions/markdown-language-features/notebook-out/**
+**/extensions/markdown-math/notebook-out/**
+**/extensions/notebook-renderers/renderer-out/index.js
+**/extensions/simple-browser/media/index.js
+**/extensions/typescript-language-features/test-workspace/**
+**/extensions/typescript-language-features/extension.webpack.config.js
+**/extensions/typescript-language-features/extension-browser.webpack.config.js
+**/extensions/typescript-language-features/package-manager/node-maintainer/**
+**/extensions/vscode-api-tests/testWorkspace/**
+**/extensions/vscode-api-tests/testWorkspace2/**
+**/fixtures/**
+**/node_modules/**
+**/out-*/**/*.js
+**/out-editor-*/**
+**/out/**/*.js
+**/src/**/dompurify.js
+**/src/**/marked.js
+**/src/**/semver.js
+**/src/typings/**/*.d.ts
+**/src/vs/*/**/*.d.ts
+**/src/vs/base/test/common/filters.perf.data.js
+**/src/vs/loader.js
+**/test/unit/assert.js
+**/test/automation/out/**
+**/typings/**
+!.vscode

.eslintignore.json

@@ -1,38 +0,0 @@
-[
-	"**/build/*/**/*.js",
-	"**/dist/**/*.js",
-	"**/extensions/**/*.d.ts",
-	"**/extensions/**/build/**",
-	"**/extensions/**/colorize-fixtures/**",
-	"**/extensions/css-language-features/server/test/pathCompletionFixtures/**",
-	"**/extensions/html-language-features/server/lib/jquery.d.ts",
-	"**/extensions/html-language-features/server/src/test/pathCompletionFixtures/**",
-	"**/extensions/ipynb/notebook-out/**",
-	"**/extensions/markdown-language-features/media/**",
-	"**/extensions/markdown-language-features/notebook-out/**",
-	"**/extensions/markdown-math/notebook-out/**",
-	"**/extensions/notebook-renderers/renderer-out/index.js",
-	"**/extensions/simple-browser/media/index.js",
-	"**/extensions/typescript-language-features/test-workspace/**",
-	"**/extensions/typescript-language-features/extension.webpack.config.js",
-	"**/extensions/typescript-language-features/extension-browser.webpack.config.js",
-	"**/extensions/typescript-language-features/package-manager/node-maintainer/**",
-	"**/extensions/vscode-api-tests/testWorkspace/**",
-	"**/extensions/vscode-api-tests/testWorkspace2/**",
-	"**/fixtures/**",
-	"**/node_modules/**",
-	"**/out-*/**/*.js",
-	"**/out-editor-*/**",
-	"**/out/**/*.js",
-	"**/src/**/dompurify.js",
-	"**/src/**/marked.js",
-	"**/src/**/semver.js",
-	"**/src/typings/**/*.d.ts",
-	"**/src/vs/*/**/*.d.ts",
-	"**/src/vs/base/test/common/filters.perf.data.js",
-	"**/src/vs/loader.js",
-	"**/test/unit/assert.js",
-	"**/test/automation/out/**",
-	"**/typings/**",
-	"!.vscode"
-]

build/azure-pipelines/product-build.yml

@@ -193,7 +193,7 @@ extends:
       eslint:
         enabled: true
         enableExclusions: true
-        exclusionsFilePath: $(Build.SourcesDirectory)/.eslintignore
+        exclusionsFilePath: $(Build.SourcesDirectory)/.eslint-ignore
       sourceAnalysisPool: 1es-windows-2022-x64
       createAdoIssuesForJustificationsForDisablement: false
     containers:

build/filters.js

@@ -197,7 +197,10 @@ module.exports.eslintFilter = [
 	'**/*.cjs',
 	'**/*.mjs',
 	'**/*.ts',
-	...require(join(__dirname, '..', '.eslintignore.json'))
+	...readFileSync(join(__dirname, '..', '.eslint-ignore'))
+		.toString()
+		.split(/\r\n|\n/)
+		.filter(line => line && !line.startsWith('#'))
 		.map(line => line.startsWith('!') ? line.slice(1) : `!${line}`)
 ];
 

eslint.config.cjs

@@ -11,7 +11,10 @@ const stylisticTs = require('@stylistic/eslint-plugin-ts');
 const pluginHeader = require('eslint-plugin-header');
 pluginHeader.rules.header.meta.schema = false;
 
-const ignores = JSON.parse(fs.readFileSync(path.join(__dirname, '.eslintignore.json'), 'utf8').toString());
+const ignores = fs.readFileSync(path.join(__dirname, '.eslint-ignore'), 'utf8')
+	.toString()
+	.split(/\r\n|\n/)
+	.filter(line => line && !line.startsWith('#'));
 
 module.exports = tseslint.config(
 	{