Skip to content
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

ESLint config based on Standard #2

Open
kouts opened this issue Oct 31, 2024 · 2 comments
Open

ESLint config based on Standard #2

kouts opened this issue Oct 31, 2024 · 2 comments

Comments

@kouts
Copy link

kouts commented Oct 31, 2024

Hey @gvre thanks for this great skeleton!
WDYT about using an ESLint config that's based on Standard?
I've opened a PR that adds such a config, it will tidy up the package.json a lot.

@gvre
Copy link
Owner

gvre commented Nov 2, 2024

Hey @kouts! Thank you for your comments and the PR. I didn't know about Standard, so I'll take a look at it.

@kouts
Copy link
Author

kouts commented Nov 4, 2024

So basically Standard is an ESLint preset that you usually put on top of ESLint recommended. That's the approach I took in https://github.com/kouts/eslint-config: start with ESLint recommended and then put Standard on top. You can check the Standard rules in detail here https://github.com/standard/eslint-config-standard/blob/master/src/index.ts

I opened the PR because I found that kouts/eslint-config shared the same ESLInt preferences as the ones you originally set in the project, but you can also override any rule you don't like. I'm maintaining it regularly so you won't have to bother about npm packages that need updating etc😄 .

Anyway, let me know and I can close the PR if that doesn't sound like a good idea. In any case, I would definitely recommend creating your own shareable ESLint config, as sooner or later you're going to need it in other projects 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants