-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
Where is repo for eslint-config-rackt exists now? #1509
Comments
I believe @mjackson removed it, as we're going to start centralizing on |
👍 Ya, I removed the repo because nobody was maintaining it. Linting is up to individual package maintainers now. Semi-colons!!!1!! |
Great, thanks for clarification! So, semicolon are now in favour? I am confused. All of the |
It’s a style Ryan and Michael chose for |
@gaearon I am happy not worrying about style at all. I like adopting code style from projects I am contributing to or grabbing code from. By this way, having preset like |
We were using |
@gaearon I remember you saying you don't use semicolons because of some team coding style, but |
I think I just clarified this above: #1509 (comment) |
@gaearon I see, great, thank you for clarification. 👍 |
Is there any chance of turning the choices from something like react-quick-start into a reactjs maintained style guide? Airbnb seems like a good place to start, but they're committed to doing their own thing, not necessarily going where the community is going.
|
FWIW I just use eslint-plugin-react's "recommended" settings these days and go from there. You can see how I do it based on some of my more recent work: |
@battaile if there's anything (besides the jsx thing, which has been discussed at length) you think the community is doing differently than the airbnb guide, please feel free to contribute at airbnb/javascript#1089 |
Suddenly all repos for ESLint presets
eslint-config-rackt
/eslint-config-reactjs
responds 404. Where are they now? Does they exists?The text was updated successfully, but these errors were encountered: