-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
Enable eslint-comments/require-description
#704
Conversation
Any new rules that can be added? |
Kinda weird, there are no further such comments. The errors you can see appear on random lines 🤔 |
Not sure I want more rules from that plugin. It has not seen any activity for years: https://github.com/mysticatea/eslint-plugin-eslint-comments |
Still worth merging this? |
No, I don't think so. The plugin is unmaintained, and while I think it could be useful to document when you ignore, it's going to be super annoying in practice and cause a huge amount of churn. I'm happy to merge this PR with just the ignore comment explanations though. |
Heh, this causes XO itself to be reported 😃