Merge pull request #394 from penge/eslint

Use more strict linter

.eslintignore

@@ -1,3 +1,4 @@
 out/*
 jest.config.js
 build.js
+set-client-id.js

.eslintrc

@@ -1,20 +1,41 @@
 {
   "root": true,
-  "parser": "@typescript-eslint/parser",
-  "plugins": [
-    "@typescript-eslint"
-  ],
   "extends": [
-    "eslint:recommended",
-    "plugin:@typescript-eslint/recommended"
+    "airbnb",
+    "airbnb-typescript"
   ],
+  "parserOptions": {
+    "project": "./tsconfig.json"
+  },
   "env": {
     "node": true
   },
+  "settings": {
+    "react": {
+      "version": "18"
+    }
+  },
   "rules": {
     "indent": ["error", 2],
-    "quotes": ["error", "double"],
+    "quotes": ["error", "double", { "avoidEscape": true }],
     "semi": ["error", "always"],
-    "no-multi-spaces": ["error", { "ignoreEOLComments": true }]
+    "no-multi-spaces": ["error", { "ignoreEOLComments": true }],
+    "max-len": ["error", { "code": 150 }],
+    "no-alert": "off",
+    "no-console": "off",
+    "implicit-arrow-linebreak": ["off"],
+    "react/react-in-jsx-scope": ["off"],
+    "react/require-default-props": ["off"],
+    "react/function-component-definition": [
+      "error",
+      {
+        "namedComponents": "arrow-function",
+        "unnamedComponents": "arrow-function"
+      }
+    ],
+    "jsx-a11y/no-autofocus": ["off"],
+    "jsx-a11y/click-events-have-key-events": ["off"],
+    "jsx-a11y/no-static-element-interactions": ["off"],
+    "@typescript-eslint/quotes": ["error", "double", { "avoidEscape": true }]
   }
 }

.gitignore

@@ -1,4 +1,4 @@
 .DS_Store
+.vscode
 node_modules/
 out/
-

env/set-client-id.js

@@ -8,7 +8,7 @@ if (!clientId) {
   return;
 }
 
-const fs = require("fs"); // eslint-disable-line
+const fs = require("fs");
 const manifestContent = JSON.parse(fs.readFileSync(manifestPath, "utf8"));
 manifestContent.oauth2.client_id = clientId;
 fs.writeFileSync(manifestPath, JSON.stringify(manifestContent, null, 2));