cleanup for eslint configs (#23)

- fixed dotenv module resolver
- organized eslint rules
- added react-hooks/ rules

.eslintrc.json

@@ -21,7 +21,7 @@
     "ecmaVersion": 12,
     "sourceType": "module"
   },
-  "plugins": ["react", "@typescript-eslint", "jest"],
+  "plugins": ["react", "@typescript-eslint", "jest", "react-hooks"],
   "settings": {
     "import/resolver": {
       "typescript": {},
@@ -33,33 +33,7 @@
   },
   "rules": {
     "@typescript-eslint/no-use-before-define": ["error"],
-    "no-use-before-define": "off",
-    "react/jsx-props-no-spreading": "off",
-    "no-console": ["error", { "allow": ["warn"] }],
-    "object-curly-newline": "off",
     "arrow-parens": [2, "as-needed"],
-    "no-unused-expressions": [
-      "error",
-      {
-        "allowShortCircuit": true,
-        "allowTernary": true
-      }
-    ],
-    "jest/no-disabled-tests": "warn",
-    "jest/no-focused-tests": "error",
-    "jest/no-identical-title": "error",
-    "jest/prefer-to-have-length": "warn",
-    "jest/valid-expect": "error",
-    "no-restricted-imports": [
-      "error",
-      {
-        "patterns": ["../*"]
-      }
-    ],
-    "react/jsx-filename-extension": [
-      1,
-      { "extensions": [".test.js", ".tsx", ".ts"] }
-    ],
     "import/extensions": [
       "error",
       "ignorePackages",
@@ -88,6 +62,34 @@
           "caseInsensitive": true
         }
       }
+    ],
+    "jest/no-disabled-tests": "warn",
+    "jest/no-focused-tests": "error",
+    "jest/no-identical-title": "error",
+    "jest/prefer-to-have-length": "warn",
+    "jest/valid-expect": "error",
+    "no-console": ["error", { "allow": ["warn"] }],
+    "no-restricted-imports": [
+      "error",
+      {
+        "patterns": ["../*"]
+      }
+    ],
+    "no-unused-expressions": [
+      "error",
+      {
+        "allowShortCircuit": true,
+        "allowTernary": true
+      }
+    ],
+    "no-use-before-define": "off",
+    "object-curly-newline": "off",
+    "react-hooks/exhaustive-deps": "warn",
+    "react-hooks/rules-of-hooks": "error",
+    "react/jsx-props-no-spreading": "off",
+    "react/jsx-filename-extension": [
+      1,
+      { "extensions": [".test.js", ".tsx", ".ts"] }
     ]
   }
 }

babel.config.js

@@ -8,7 +8,7 @@ module.exports = function (api) {
       [
         'module:react-native-dotenv',
         {
-          moduleName: '@env',
+          moduleName: "react-native-dotenv",
           path: '.env',
           safe: true,
         },

env.d.ts

@@ -0,0 +1,8 @@
+declare module 'react-native-dotenv' {
+  export const API_KEY: any;
+  export const AUTH_DOMAIN: any;
+  export const PROJECT_ID: any;
+  export const STORAGE_BUCKET: any;
+  export const MESSAGING_SENDER_ID: any;
+  export const APP_ID: any;
+}

src/database/firebase.tsx

@@ -1,12 +1,12 @@
+import firebase from 'firebase';
 import {
   API_KEY,
   AUTH_DOMAIN,
   PROJECT_ID,
   STORAGE_BUCKET,
   MESSAGING_SENDER_ID,
   APP_ID,
-} from '@env';
-import firebase from 'firebase';
+} from 'react-native-dotenv';
 import 'firebase/firestore';
 
 import { Tree, Comment, Additional } from '@types';