-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
fix: skip add plugin when already imported #70
fix: skip add plugin when already imported #70
Conversation
The committers listed above are authorized under a signed CLA. |
Hi @syi0808!, thanks for the Pull Request The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.
To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page. Read more about contributing to ESLint here |
Thanks for the contribution. We need you to fill out a bug report before we can consider this. |
I fill out a bug report and connect this pr. thank you! |
@syi0808 we also need you to sign the CLA. Please see the comment posted by the bot. |
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.
Changes LGTM. Just need the CLA signed to merge.
I'm sorry to have made you check many times. I signed CLA now. thank you! |
No worries. Thank you! |
Prerequisites checklist
What is the purpose of this pull request?
prevent syntax error which threw by duplicate import fixupPluginRules because of multiple compat needs plugins.
What changes did you make? (Give an overview)
Related Issues
fixes #72
Is there anything you'd like reviewers to focus on?