thoughtbot / eslint-config

A sharable ESLint configuration that enforces thoughtbot’s JavaScript guides.
https://thoughtbot.com
MIT License
10 stars 1 forks source link

Enhance the import order #4

Closed pangolingo closed 1 year ago

pangolingo commented 1 year ago

This is fairly opinionated. I'd be happy to leave the grouping as is, instead of grouping parent and sibling imports.

This import rule is only good if you have an autoformatter that can do it for you (VSCode does it for me). Otherwise it's very painful.

pangolingo commented 1 year ago

I'm going to close this out because I want to use the Typescript autoformatter's defaults and not force folks to use eslint unnecessarily. In the future if I feel like spaces between import groups or alphabetization would provide a big benefit, I may reopen this PR.

stevehanson commented 1 year ago

@pangolingo definitely open to continuing the discussion!