silverwind

To add one data point, here is the rule's performance on [this repo](https://github.com/go-gitea/gitea). ``` $ TIMING=1 make lint-js npx eslint --color --max-warnings=0 --ext js,ts,vue web_src/js tools *.js *.ts *.cjs tests/e2e...

I should have noted that those 20x gains were for `eslint-plugin-import`, maybe this plugin things differently and there are no gains to be had. Still I guess I could not...

Running https://github.com/un-ts/eslint-plugin-import-x/issues/113#issuecomment-2682085698 again with updated dependencies, it seems much improved: ``` $ TIMING=1 make lint-js npx eslint --color --max-warnings=0 --ext js,ts,vue web_src/js tools *.js *.ts *.cjs tests/e2e Rule | Time...

Yes I will. Haven't gotten around to it yet, but it's on my list 😉.

So I checked and did not find the `@rtsao/scc` dependency in this repo which was used for SCC at some point in the past. With that said, there is no...

I can't judge knip until it actually works for me (https://github.com/webpro-nl/knip/issues/725).

And no, I don't think it's a good idea to deprecate a lint rule because that tool exists. The rule is focused on just finding unused code while that tool...

For my personal config, I will just disable `no-unused-modules` because I find the value it provides to be rather minimal and I end up fighting it with `eslint-disable` comments more...

> For my personal config, I will just disable `no-unused-modules` because I find the value it provides to be rather minimal and I end up fighting it with `eslint-disable` comments...

Yes, I need to investigate knip further, now that https://github.com/webpro-nl/knip/issues/725 is fixed it works on a basic level, but still has a lot of false-positive issues for the other stuff...