@marlas/config 0.1.0 → 0.2.2

package/README.md

@@ -1,5 +1,5 @@
-# `@marlas/configs`
+# `@marlas/config`
 
 Configs for tools:
 
-- oxlint - `extends: ['@marlas/configs/oxlint']`
+- oxlint - `extends: ['./node_modules/@marlas/config/src/oxlint.json']`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@marlas/config",
-  "version": "0.1.0",
+  "version": "0.2.2",
   "description": "Various tools default configs to extend in projects",
   "author": "Marcin Lasak",
   "license": "MIT",
@@ -26,10 +26,7 @@
     "oxlint",
     "linter"
   ],
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/marlas/configs.git"
-  },
+  "repository": "github:marlass/configs",
   "publishConfig": {
     "access": "public"
   },

package/src/oxlint.json

@@ -6,7 +6,7 @@
         "pedantic": "error",
         "perf": "error",
         "restriction": "error",
-        "style": "off",
+        "style": "error",
         "nursery": "error"
     },
     "plugins": [
@@ -31,8 +31,28 @@
         "eslint/max-lines": "off",
         "eslint/max-statements": "off",
         "eslint/max-params": "off",
+        // Sorting keys is bad idea. Very often you want to group by purpose
+        "eslint/sort-keys": "off",
+        // Sorting imports is formatter responsibility, not the linter.
+        "eslint/sort-imports": "off",
+        // This doesn't really matter. Sometimes you want to use arrow function, sometimes you want to use function declaration.
+        "eslint/func-style": "off",
+        // Sometimes single character variables are pretty useful
+        "eslint/id-length": "off",
+        // Ternary are fine
+        "eslint/no-ternary": "off",
+        // Too restrictive. There are number of patterns where this is useful
+        "eslint/init-declarations": "off",
+        // Object.assign is more memory efficient than spread operator
+        "eslint/prefer-object-spread": "off",
+        // 0 is often use to check presence of something and -1/1 are often used in sorting
+        "eslint/no-magic-numbers": ["error", { "ignore": [0, 1, -1] }],
         // undefined has it's uses
         "eslint/no-undefined": "off",
+        // Usefull for modifications later on without need to wrap in parentheses
+        "eslint/arrow-body-style": ["error", "always"],
+        // Duplicates are fine for type imports
+        "eslint/no-duplicate-imports": ["error", { "allowSeparateTypeImports": true }],
         // async/await is supported almost everywhere now
         "oxc/no-async-await": "off",
         // React import are handled by the build tools for ergonomics
@@ -49,6 +69,18 @@
         // Enforce .tsx extension for React files
         "react/jsx-filename-extension": ["error", {
             "extensions": [".tsx"]
-        }]
+        }],
+        // Bad idea for complexity metric
+        "react/jsx-max-depth": "off",
+        // Doesn't matter where the export is
+        "import/exports-last": "off",
+        // Grouping exports only require extra effort
+        "import/group-exports": "off",
+        // Named exports are the most ergonomic and should be used as default
+        "import/no-named-export": "off",
+        // Numbers are easier to spot than strings
+        "vitest/prefer-called-once": "off",
+        // These are often usefull for eg. starting server in tests
+        "jest/no-hooks": "off"
     }
 }