-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Clean up @babel/eslint-plugin #10756
Conversation
351d5ca
to
8408c94
Compare
8408c94
to
74a5dec
Compare
Co-Authored-By: Huáng Jùnliàng <jlhwung@gmail.com>
Thanks for the review! Ready for re-review. |
"babel/semi": 1, | ||
"babel/no-unused-expressions": 1, | ||
"babel/valid-typeof": 1 | ||
"babel/new-cap": "error", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The rule name should be prefixed by the plugin name: i.e. @babel/eslint-plugin/new-cap
. Could you check if @babel/new-cap
works?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure - my intention isn't to update everything here, but to just get a baseline so I can start working. I was imaging that a lot of this README is going to be changed anyway, since I'm hoping to remove the deprecated rules and rethink how we keep this up-to-date with ESLint core rules.
Preliminary housekeeping tasks for @babel/eslint-plugin.