-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Is my custom eslint shareable config good like this or should I change it? #38
Comments
@rfgamaral They're same anyway. But I'm using For sharable configuration, you can check the source codes of https://github.com/1stG/configs/tree/master/packages/eslint-config as reference. |
@JounQin I asked because I'm having a similar issue to #37 and I can't understand what's wrong with my configuration. I thought I could be doing something wrong with my ESLint config package. I'll try to create a minimal reproducible repository tomorrow and post on the other issue, maybe you can understand what I'm doing wrong... |
@rfgamaral That's fine. I'm wondering if my PR helps? |
I looked at that but there are so many changes that I'm not sure what's the bit that actually fixes the problem... |
@rfgamaral The key difference is that we should only have one |
@JounQin Actually, I'm not sure I have the same problem as #37. I mean, I get this error in VS Code with the ESLint extension: Maybe my problem is actually in the ESLint extension and not the configuration...? |
@rfgamaral Sorry, no idea about this. |
That's fine, no worries. Thanks for your support :) Anyway, I've posted this if you're interested to follow up. |
I have created this shareable ESLint configuration and I'm wondering if the following:
Is the same as:
?
If not, could you be so kind in explaining the differences? Should I change this?
The text was updated successfully, but these errors were encountered: