Tweak eslint config to better handle json/md, remove compile-tests script

jaggederest · jaggederest · commit 12b0124b74cc · 2025-06-18T14:38:06.000-07:00
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -17,26 +17,20 @@
 	],
 	"overrides": [
 		{
-			"files": ["*.md"],
-			"parser": "markdown-eslint-parser",
+			"files": ["*.ts"],
 			"rules": {
-				"@typescript-eslint/require-await": "off"
+				"require-await": "off",
+				"@typescript-eslint/require-await": "error"
 			}
 		},
 		{
+			"extends": ["plugin:package-json/legacy-recommended"],
 			"files": ["*.json"],
-			"parser": "jsonc-eslint-parser",
-			"rules": {
-				"@typescript-eslint/require-await": "off"
-			}
+			"parser": "jsonc-eslint-parser"
 		},
 		{
-			"extends": ["plugin:package-json/legacy-recommended"],
-			"files": ["package.json"],
-			"parser": "jsonc-eslint-parser",
-			"rules": {
-				"@typescript-eslint/require-await": "off"
-			}
+			"files": ["*.md"],
+			"parser": "markdown-eslint-parser"
 		}
 	],
 	"rules": {
@@ -73,9 +67,7 @@
 					"sublings_only": true
 				}
 			}
-		],
-		"require-await": "off",
-		"@typescript-eslint/require-await": "error"
+		]
 	},
 	"ignorePatterns": ["out", "dist", "**/*.d.ts"]
 }
diff --git a/package.json b/package.json
@@ -19,13 +19,12 @@
 	"main": "./dist/extension.js",
 	"scripts": {
 		"build": "webpack",
-		"compile-tests": "tsc -p . --outDir out",
 		"fmt": "prettier --write .",
 		"lint": "eslint . --ext ts,md,json",
 		"lint:fix": "yarn lint --fix",
 		"package": "webpack --mode production --devtool hidden-source-map",
 		"package:prerelease": "npx vsce package --pre-release",
-		"pretest": "yarn run compile-tests && yarn run build && yarn run lint",
+		"pretest": "tsc -p . --outDir out && yarn run build && yarn run lint",
 		"test": "vitest",
 		"test:ci": "CI=true yarn test",
 		"test:integration": "vscode-test",

Original file line number	Diff line number	Diff line change
`@@ -17,26 +17,20 @@`
`17`	`17`	`],`
`18`	`18`	`"overrides": [`
`19`	`19`	`{`
`20`		`- "files": ["*.md"],`
`21`		`- "parser": "markdown-eslint-parser",`
	`20`	`+ "files": ["*.ts"],`
`22`	`21`	`"rules": {`
`23`		`- "@typescript-eslint/require-await": "off"`
	`22`	`+ "require-await": "off",`
	`23`	`+ "@typescript-eslint/require-await": "error"`
`24`	`24`	`}`
`25`	`25`	`},`
`26`	`26`	`{`
	`27`	`+ "extends": ["plugin:package-json/legacy-recommended"],`
`27`	`28`	`"files": ["*.json"],`
`28`		`- "parser": "jsonc-eslint-parser",`
`29`		`- "rules": {`
`30`		`- "@typescript-eslint/require-await": "off"`
`31`		`- }`
	`29`	`+ "parser": "jsonc-eslint-parser"`
`32`	`30`	`},`
`33`	`31`	`{`
`34`		`- "extends": ["plugin:package-json/legacy-recommended"],`
`35`		`- "files": ["package.json"],`
`36`		`- "parser": "jsonc-eslint-parser",`
`37`		`- "rules": {`
`38`		`- "@typescript-eslint/require-await": "off"`
`39`		`- }`
	`32`	`+ "files": ["*.md"],`
	`33`	`+ "parser": "markdown-eslint-parser"`
`40`	`34`	`}`
`41`	`35`	`],`
`42`	`36`	`"rules": {`
`@@ -73,9 +67,7 @@`
`73`	`67`	`"sublings_only": true`
`74`	`68`	`}`
`75`	`69`	`}`
`76`		`- ],`
`77`		`- "require-await": "off",`
`78`		`- "@typescript-eslint/require-await": "error"`
	`70`	`+ ]`
`79`	`71`	`},`
`80`	`72`	`"ignorePatterns": ["out", "dist", "*/.d.ts"]`
`81`	`73`	`}`