eslint-config-beslogic 5.1.2 → 5.2.1

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 5.2.1
+
+- Replaced `react/jsx-sort-props` with `@stylistic/jsx-sort-props`
+- Configured `@stylistic/jsx-sort-props` to prefer reserved keywords (then) shorthands first, and prefer multilines last.
+
 ## 5.1.2
 
 - Move `optionalDependencies` to `peerDependencies` marked with `peerDependenciesMeta.optional`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-beslogic",
-  "version": "5.1.2",
+  "version": "5.2.1",
   "description": "ESLint rules, plugins and configs used at Beslogic",
   "main": "index.js",
   "type": "module",

package/react.mjs

@@ -92,6 +92,18 @@ export default defineConfig(
       // Using only max-lines and not max-lines-per-function as most of the react code lives in functions.
       "max-lines-per-function": "off",
 
+      /*
+       * Stylistic overrides https://eslint.style/rules
+       */
+      "react/jsx-sort-props": "off",
+      "@stylistic/jsx-sort-props": [
+        "error",
+        // Keeping special props first, and not hiding props under big multiline
+        // both help a lot with readability and semantics
+        // Still keeping alphabetical sorting for automatic consistency
+        { "reservedFirst": true, "shorthandFirst": true, "multiline": "last" }
+      ],
+
       /*
        * react overrides (https://github.com/yannickcr/eslint-plugin-react#list-of-supported-rules)
        */

package/stylistic.mjs

@@ -215,8 +215,7 @@ export default defineConfig(
         { "allow": "single-line" }
       ],
       "@stylistic/jsx-pascal-case": "error",
-      "@stylistic/jsx-self-closing-comp": "error",
-      "@stylistic/jsx-sort-props": "error"
+      "@stylistic/jsx-self-closing-comp": "error"
     }
   }
 )