chore: add eslint-rule to disallow constructed values in context valu…

…es (#297) As discovered in #285 there was a problem with excessive re-renders due to context values not being memoized across renders. The now added eslint-rule `react/jsx-no-constructed-context-values` catches these sort of mistakes and reports them as error. Co-authored-by: Rodrigo Faria <[email protected]>
visgl · Apr 3, 2024 · 495ca24 · 495ca24
1 parent 833933a
commit 495ca24
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -43,7 +43,8 @@ module.exports = {
 
         'no-undef': 'off',
         'react-hooks/rules-of-hooks': 'error',
-        'react-hooks/exhaustive-deps': 'warn',
+        'react-hooks/exhaustive-deps': 'error',
+        'react/jsx-no-constructed-context-values': 'error',
 
         // We use function hoisting to put exports at top of file
         '@typescript-eslint/no-use-before-define': 'off',