-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
resolve eslint config and plugins manually #38
base: master
Are you sure you want to change the base?
Conversation
Well shoot. this seems to not work if there are any plugins specified in a local .eslintrc.json file. Its trying to resolve the plugins relative to the standardx |
Solved this by deleting my old .eslintrc.js re-running yarn lint from the command line. |
Any progress on merging this? We are still stuck with #27 |
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.
I approve of these changes. It works for me.
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.
Solved this by deleting my old .eslintrc.js re-running yarn lint from the command line.
Next automatically added the new eslintrc.json file
did this and it worked great
Might be possible to fix with the new FlatConfig .. https://eslint.org/blog/2022/08/new-config-system-part-2/ |
fixes #27
What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[💋] Bug fix
[ ] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
resolvePluginsRelativeTo: true
to eslint config to ensure plugins resolve correctly.Which issue (if any) does this pull request address?
#27
Is there anything you'd like reviewers to focus on?