eslint-config: consider consuming "simple-import-sort" eslint plugin

Hi all,

please consider within the RW linting machinery to ship by default simple-import-sort eslint plugin as would make code objectively more readable out of the box (again IMHO), and (more debatable) also eventually the json one.

current workaround is to use (in top-level’s package.json)

"devDependencies": {
    "@redwoodjs/core": "^0.19.2-canary.45",
    "eslint-plugin-json": "2.1.2",
    "eslint-plugin-simple-import-sort": "5.0.3"
  },
  "eslintConfig": {
    "extends": [
      "@redwoodjs/eslint-config"
    ],
    "plugins": [
      "json",
      "simple-import-sort"
    ],
    "rules": {
      "json/*": [
        "error",
        "allowComments"
      ],
      "simple-import-sort/sort": "error"
    }
  }, 

but IMHO would be really nice if stuff coming from generators would come ‘nice’ by default.

All the Best,

António

P.S. ironically “import sorting” is already referred as a feature of eslint-config here

[peterp]

Hey there - we actually had something like this in our ESLint configuration before, but we removed it because it made it more difficult for people to contribute, and it didn’t work with directory named modules.

Now that we have the babel-module for directory-named-modules we could probably try this again.

[AntonioMeireles]

Oops, just a quick side note:

there was a typo im my original copy & paste. no need at all to hard-lock the versions so it would be something along …

“eslint-plugin-json”: “^2.1.2”, “eslint-plugin-simple-import-sort”: “^5.0.3”

&& thanks!

António