feat(react): enforce using ternary for conditional rendering

BREAKING CHANGE: conditional rendering must be done via a ternary operator
RimacTechnology · Jul 7, 2022 · 3c79f96 · 3c79f96
1 parent f6afc5a
commit 3c79f96
Show file tree

Hide file tree

Showing 3 changed files with 261 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -44,10 +44,10 @@
         "eslint-plugin-import": "^2.26.0",
         "eslint-plugin-jest": "^26.5.3",
         "eslint-plugin-jest-formatting": "^3.1.0",
-        "eslint-plugin-mobx": "^0.0.8",
+        "eslint-plugin-mobx": "^0.0.9",
         "eslint-plugin-more": "^1.0.5",
         "eslint-plugin-promise": "^6.0.0",
-        "eslint-plugin-react": "^7.29.3",
+        "eslint-plugin-react": "^7.30.1",
         "eslint-plugin-react-hooks": "^4.3.0",
         "eslint-plugin-sonarjs": "^0.12.0",
         "eslint-plugin-sort-class-members": "^1.14.1",

diff --git a/react.js b/react.js
@@ -74,6 +74,10 @@ module.exports = {
         'react/jsx-no-comment-textnodes': 'error',
         'react/jsx-no-duplicate-props': 'error',
         'react/jsx-no-useless-fragment': ['error', { 'allowExpressions': true }],
+        "react/jsx-no-leaked-render": [
+            "error",
+            { "validStrategies": ["ternary"] }
+        ],
         'react/jsx-one-expression-per-line': 'error',
         'react/jsx-pascal-case': 'error',
         'react/jsx-props-no-multi-spaces': 'error',