fix: remove only-warn plugin

Author: aminya

spec/test.mts

@@ -1,8 +1,9 @@
 import { command } from "execa"
 import { resolve, dirname, join } from "path"
 import packageJson from "../package.json" with { type: "json" }
-import * as gitly from "./gitly.js"
-const { download, extract } = gitly
+// @ts-ignore
+import gitly from "./gitly.js"
+const { download, extract } = gitly as typeof import("./gitly.js")
 import fs from "fs-extra"
 const { pathExists, ensureDir, remove } = fs
 import { fileURLToPath } from "url"

src/astro.cts

@@ -1,7 +1,6 @@
 import type { Linter } from "eslint"
 import { pluginImportAstroRulesExtra } from "./plugin-import-rules.cjs"
 import astroPlugin from "eslint-plugin-astro"
-import * as onlyWarnPlugin from "eslint-plugin-only-warn"
 import astroParser from "astro-eslint-parser"
 
 export const astroConfig: Linter.Config[] = [
@@ -21,7 +20,6 @@ export const astroConfig: Linter.Config[] = [
     },
     plugins: {
       astro: astroPlugin,
-      "only-warn": onlyWarnPlugin,
     },
     rules: {
       ...pluginImportAstroRulesExtra,

src/coffeescript.cts

@@ -2,7 +2,6 @@ import { pluginNodeRules } from "./plugin-node-rules.cjs"
 import { pluginImportRulesExtra } from "./plugin-import-rules.cjs"
 import type { Linter } from "eslint"
 import * as eslintPluginCoffee from "eslint-plugin-coffee"
-import * as onlyWarnPlugin from "eslint-plugin-only-warn"
 import * as nodePlugin from "eslint-plugin-node"
 import * as coffeeParser from "eslint-plugin-coffee/parser"
 
@@ -17,7 +16,6 @@ export const coffeeConfig: Linter.Config = {
   plugins: {
     coffee: eslintPluginCoffee,
     node: nodePlugin,
-    "only-warn": onlyWarnPlugin,
   },
   rules: {
     ...pluginNodeRules,

src/index-strict-react.cts

@@ -1,18 +1,3 @@
 import nonStrictConfig from "./index-react.cjs"
 
-// // remove only-warn from javascript
-// if (
-//   nonStrictConfig.plugins !== undefined &&
-//   nonStrictConfig.plugins[nonStrictConfig.plugins.length - 1] === "only-warn"
-// ) {
-//   nonStrictConfig.plugins.pop()
-// }
-
-// // remove only-warn from overrides
-// nonStrictConfig.overrides?.forEach((overrides) => {
-//   if (overrides.plugins !== undefined && overrides.plugins[overrides.plugins.length - 1] === "only-warn") {
-//     overrides.plugins.pop()
-//   }
-// })
-
 export default nonStrictConfig

src/index-strict-solid.cts

@@ -1,18 +1,4 @@
 import nonStrictConfig from "./index-solid.cjs"
 
-// // remove only-warn from javascript
-// if (
-//   nonStrictConfig.plugins !== undefined &&
-//   nonStrictConfig.plugins[nonStrictConfig.plugins.length - 1] === "only-warn"
-// ) {
-//   nonStrictConfig.plugins.pop()
-// }
-
-// // remove only-warn from overrides
-// nonStrictConfig.overrides?.forEach((overrides) => {
-//   if (overrides.plugins !== undefined && overrides.plugins[overrides.plugins.length - 1] === "only-warn") {
-//     overrides.plugins.pop()
-//   }
-// })
 
 export default nonStrictConfig

src/index-strict.cts

@@ -1,18 +1,3 @@
 import nonStrictConfig from "./index.cjs"
 
-// // remove only-warn from javascript
-// if (
-//   nonStrictConfig.plugins !== undefined &&
-//   nonStrictConfig.plugins[nonStrictConfig.plugins.length - 1] === "only-warn"
-// ) {
-//   nonStrictConfig.plugins.pop()
-// }
-
-// // remove only-warn from overrides
-// nonStrictConfig.overrides?.forEach((overrides) => {
-//   if (overrides.plugins !== undefined && overrides.plugins[overrides.plugins.length - 1] === "only-warn") {
-//     overrides.plugins.pop()
-//   }
-// })
-
 export default nonStrictConfig

src/javascript.cts

@@ -5,7 +5,6 @@ import type { Linter } from "eslint"
 import * as eslintBabelParser from "@babel/eslint-parser"
 import * as nodePlugin from "eslint-plugin-node"
 import * as importPlugin from "eslint-plugin-import"
-import * as onlyWarnPlugin from "eslint-plugin-only-warn"
 
 import js from "@eslint/js"
 
@@ -33,7 +32,7 @@ export const jsConfig: Linter.Config[] = [
     plugins: {
       node: nodePlugin,
       import: importPlugin,
-      "only-warn": onlyWarnPlugin,
+
     },
     rules: {
       ...eslintRulesExtra,

src/typescript.cts

@@ -3,7 +3,6 @@ import * as typeScriptPlugin from "typescript-eslint"
 import type { Linter } from "eslint"
 import * as importPlugin from "eslint-plugin-import"
 import * as nodePlugin from "eslint-plugin-node"
-import * as onlyWarnPlugin from "eslint-plugin-only-warn"
 import type { GlobifiedEntry } from "globify-gitignore"
 import makeSynchronous from "make-synchronous"
 import { eslintRulesExtra } from "./official-eslint-rules.cjs"
@@ -137,7 +136,6 @@ export const tsConfig: Linter.Config = {
   plugins: {
     node: nodePlugin,
     import: importPlugin,
-    "only-warn": onlyWarnPlugin,
   },
   rules: {
     ...javaScriptRules(),

tsconfig.json

@@ -15,8 +15,8 @@
     // check
     "strict": true,
     "strictNullChecks": true,
-    "noUnusedLocals": true,
-    "noUnusedParameters": true,
+    "noUnusedLocals": false,
+    "noUnusedParameters": false,
     "noImplicitReturns": true,
     "noImplicitAny": true,
     "noImplicitThis": true,