fix(eslint-config, eslint-config-react): update rules (#53)

* fix(eslint-config, eslint-config-react): update rules * add Tom910 and SuperOleg39 to CODEOWNERS Co-authored-by: o.drapeza <o.drapeza@tinkoff.ru>
tramvaijs · Jul 8, 2020 · 40cc0b1 · 40cc0b1
1 parent 898a74c
commit 40cc0b1
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -9,5 +9,5 @@
 #  More info: https://help.github.com/articles/about-codeowners/
 #
 
-* @YuliaTsareva @MarsiBarsi @waterplea @nezed
+* @YuliaTsareva @MarsiBarsi @waterplea @nezed @Tom910 @SuperOleg39
 # will be requested for review when someone opens a pull request
diff --git a/packages/eslint-config-react/index.js b/packages/eslint-config-react/index.js
@@ -48,7 +48,7 @@ module.exports = {
     'react/jsx-key': 'error',
     'react/no-direct-mutation-state': 'error',
     'react/default-props-match-prop-types': 'warn',
-    'react/require-default-props': 'warn',
+    'react/require-default-props': 'off', // default values in props destructuring is not supported
     'react/static-property-placement': ['error', 'static public field'],
 
     'react/sort-comp': [

diff --git a/packages/eslint-config/internal/typescript.js b/packages/eslint-config/internal/typescript.js
@@ -69,7 +69,9 @@ module.exports = {
         ],
         // '@typescript-eslint/no-unnecessary-qualifier': 'error', need ts config
         // '@typescript-eslint/restrict-plus-operands': 'error', need ts config
-        '@typescript-eslint/no-explicit-any': 'warn',
+        '@typescript-eslint/no-explicit-any': 'off',
+        camelcase: 'off',
+        '@typescript-eslint/camelcase': 'off',
         '@typescript-eslint/naming-convention': [
           'error',
           {
@@ -86,7 +88,7 @@ module.exports = {
           },
           {
             selector: 'typeLike',
-            format: ['PascalCase'],
+            format: ['PascalCase', 'UPPER_CASE'],
           },
           {
             selector: 'property',