fix(toolchain/eslint-config): add 'sort-imports' to javascript rules (#…

…25)
mpellegrini · Mar 18, 2024 · 0323749 · 0323749
1 parent 3c6e584
commit 0323749
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/packages/example-pkg/src/index.test.ts b/packages/example-pkg/src/index.test.ts
@@ -1,4 +1,4 @@
-import { greetUser, type User } from './index.js'
+import { type User, greetUser } from './index.js'
 
 describe('greet user tests', () => {
   const user: User = {

diff --git a/toolchain/eslint-config/configs/javascript.js b/toolchain/eslint-config/configs/javascript.js
@@ -17,6 +17,22 @@ export default defineFlatConfig([
     rules: {
       ...js.configs.recommended.rules,
 
+      /**
+       * Enforce sorted import declarations within modules.
+       *
+       * 🔧 Fixable - https://eslint.org/docs/latest/rules/sort-imports
+       */
+      'sort-imports': [
+        'error',
+        {
+          ignoreCase: false,
+          ignoreDeclarationSort: true,
+          ignoreMemberSort: false,
+          memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single'],
+          allowSeparatedGroups: false,
+        },
+      ],
+
       /**
        * Disallow empty functions.
        *