-
-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
no-unused-disable
doesn't work with flat config
#79
Comments
Hi @teppeis! 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 |
@MichaelDeBoey I've reopened there: eslint-community#133 |
@MichaelDeBoey if this repo is unmaintained, shouldn't it be updated / marked as unmaintained in the README and potentially archived? @mysticatea that might help avoid confusion if you're not intending to work on this plugin anymore. |
@IanVS Unfortunately we can't contact @mysticatea as otherwise we would have loved to transfer this repo to the new org instead |
I hope he's okay. :-/ |
This rule is implemented by patching
Linter
, which seems to be causing this problem with new flat config.The flat config allows
reportUnusedDisableDirectives
to be included in shareable config.The new RFC will also allow for more configuration flexibility (in ESLint v9 ?).
So it would be good to make this rule deprecated in the future.
The text was updated successfully, but these errors were encountered: