chore: ditch eslint (#219)

upgrading to `eslint@^9` is a pain, lets make the switch to `biome` for
linting instead.

Author: mnahkies

.eslintignore

@@ -47,7 +47,6 @@ jobs:
       - run: yarn ci-test
 
       - run: yarn ci-lint
-      - run: yarn ci-format
 
       - run: yarn integration:generate
       - run: yarn integration:validate

.eslintrc.js

@@ -3,7 +3,6 @@ node_modules
 .idea
 .nyc_output
 coverage
-.eslintcache
 .env
 .DS_Store
 

.github/workflows/ci.yml

@@ -81,8 +81,8 @@ yarn lint
 yarn format
 ```
 
-We use `biome` for formatting, and `eslint` for linting, largely sticking to the recommended rules.
-The config can be seen here [./.eslintrc.js](./.eslintrc.js)
+We use `biome` for formatting, and linting, largely sticking to the recommended rules.
+The config can be seen here [./biome.json](./biome.json)
 
 ## Testing
 

.gitignore

@@ -33,9 +33,18 @@
     }
   },
   "linter": {
-    "enabled": false,
+    "enabled": true,
     "rules": {
       "recommended": true
-    }
+    },
+    "ignore": [
+      "packages/openapi-code-generator/src/core/schemas/openapi-3.0-specification-validator.js",
+      "packages/openapi-code-generator/src/core/schemas/openapi-3.1-specification-validator.js",
+      "integration-tests/**/*",
+      "integration-tests-definitions/**/*",
+      "schemas/*.json",
+      "packages/*/package.json",
+      "package.json"
+    ]
   }
 }

CONTRIBUTING.md

@@ -1,5 +1,4 @@
-// eslint-disable-next-line @typescript-eslint/no-var-requires
-const inspector = require("inspector")
+const inspector = require("node:inspector")
 
 /**
  * @type { import('@jest/types').Config.GlobalConfig }

biome.json

@@ -1,4 +1,3 @@
-// eslint-disable-next-line @typescript-eslint/no-var-requires
 const {DefaultReporter} = require("@jest/reporters")
 
 class Reporter extends DefaultReporter {
@@ -7,6 +6,7 @@ class Reporter extends DefaultReporter {
     if (result.numFailingTests === 0 && !result.testExecError) {
       result.console = []
     } else {
+      // biome-ignore lint/style/noArguments: <explanation>
       super.printTestFileHeader(...arguments)
     }
   }

jest.config.js

@@ -20,32 +20,26 @@
     "clean": "lerna run clean --stream",
     "docs:generate": "./scripts/generate-docs.sh",
     "refresh": "./scripts/refresh-data.sh",
-    "lint": "eslint . --cache --report-unused-disable-directives --fix",
+    "lint": "biome lint --apply .",
     "format": "biome check --apply .",
     "build": "node ./scripts/generate-ajv-validator.js && lerna run build  --stream --scope '@nahkies/*'",
     "test": "NODE_OPTIONS=--experimental-vm-modules jest",
     "integration:generate": "node ./scripts/generate.mjs",
     "integration:validate": "lerna run validate --stream",
     "ci-build": "yarn build",
     "ci-test": "NODE_OPTIONS=--experimental-vm-modules jest --coverage",
-    "ci-lint": "eslint . --cache --report-unused-disable-directives",
-    "ci-format": "biome ci .",
+    "ci-lint": "biome ci .",
     "ci-pipeline": "./scripts/ci-pipeline.sh",
     "publish:alpha": "./scripts/publish.alpha.sh",
     "publish:release": "./scripts/publish.release.sh"
   },
   "devDependencies": {
-    "@biomejs/biome": "^1.7.3",
+    "@biomejs/biome": "1.7.3",
     "@swc/core": "^1.5.7",
     "@swc/jest": "^0.2.36",
     "@tsconfig/node20": "^20.1.4",
     "@tsconfig/strictest": "^2.0.5",
     "@types/node": "^20.12.12",
-    "@typescript-eslint/eslint-plugin": "^7.10.0",
-    "@typescript-eslint/parser": "^7.10.0",
-    "eslint": "^8.57.0",
-    "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-jest": "^28.5.0",
     "husky": "^9.0.11",
     "jest": "^30.0.0-alpha.3",
     "lerna": "^8.1.3",
@@ -61,7 +55,7 @@
   ],
   "lint-staged": {
     "*.{ts,js,tsx,jsx}": [
-      "eslint --cache --report-unused-disable-directives --fix",
+      "yarn biome lint --apply",
       "yarn biome check --apply --no-errors-on-unmatched"
     ],
     "*.md": [