no-unused-disable is redundant

[CruseCtrl]

In the docs it says

TIP
The eslint-comments/no-unused-disable rule has the same effect as --report-unused-disable-directives option. However, the eslint-comments/no-unused-disable rule is relatively useful since it can be configured in shareable configs.

While this is true, ESLint also provides the reportUnusedDisableDirectives config option, which can be configured in shareable configs: https://eslint.org/docs/latest/use/configure/rules#report-unused-eslint-disable-comments

And the default value of that option was changed to true a few weeks ago, so will probably be included as a default setting with ESLint v9: eslint/eslint#17879

I propose that no-unused-disable should be deprecated, and removed in a future release

[CruseCtrl]

Here's the link to the ESLint v9 docs: https://eslint.org/docs/next/use/configure/configuration-files#reporting-unused-disable-directives

This setting defaults to "warn".

It also has auto-fix, so running eslint --fix will automatically remove any unused disable directives by default in ESLint v9

[MichaelDeBoey]

Hi @CruseCtrl!

Since this repo is unmaintained, you might want to re-open this issue in the @eslint-community fork https://github.com/eslint-community/eslint-plugin-eslint-comments

For more info about why we created this organization, you can read https://eslint.org/blog/2023/03/announcing-eslint-community-org

[CruseCtrl]

Thanks, I didn't realise this was unmaintained. I'll maybe raise an issue on the forked one