add eslint; fix undeclared vars, run pre-commit on all files

Author: DZGoldman

.eslintrc.js

@@ -0,0 +1,44 @@
+module.exports = {
+  env: {
+    commonjs: true,
+    es6: true,
+    node: true,
+  },
+  plugins: ['prettier'],
+  extends: [
+    'eslint:recommended',
+    'plugin:prettier/recommended', // Enables eslint-plugin-prettier and displays prettier errors as ESLint errors. Make sure this is always the last configuration in the extends array.
+  ],
+  parserOptions: {
+    ecmaVersion: 2018, // Allows for the parsing of modern ECMAScript features
+    sourceType: 'module', // Allows for the use of imports
+  },
+  rules: {
+    'prettier/prettier': 'error',
+    'no-unused-vars': 'warn',
+    'prefer-const': [2, { destructuring: 'all' }],
+    'object-curly-spacing': ['error', 'always'],
+  },
+  overrides: [
+    {
+      files: ['*.ts', '*.tsx'],
+      parser: '@typescript-eslint/parser',
+      extends: [
+        'eslint:recommended',
+        'plugin:@typescript-eslint/recommended',
+        'prettier/@typescript-eslint',
+        'plugin:prettier/recommended',
+      ],
+      plugins: ['@typescript-eslint', 'prettier'],
+      rules: {
+        'no-empty-pattern': 'warn',
+        'prettier/prettier': ['error', { singleQuote: true }],
+        '@typescript-eslint/member-delimiter-style': ['off'],
+        '@typescript-eslint/no-explicit-any': ['warn'],
+        '@typescript-eslint/no-use-before-define': ['off'],
+        '@typescript-eslint/no-non-null-assertion': ['off'],
+        '@typescript-eslint/ban-ts-comment': ['warn'],
+      },
+    },
+  ],
+}

.prettierrc.js

@@ -0,0 +1,22 @@
+module.exports = {
+  semi: false,
+  trailingComma: 'es5',
+  singleQuote: true,
+  printWidth: 80,
+  tabWidth: 2,
+  arrowParens: 'avoid',
+  bracketSpacing: true,
+  overrides: [
+    {
+      files: '*.sol',
+      options: {
+        printWidth: 100,
+        tabWidth: 4,
+        useTabs: false,
+        singleQuote: false,
+        bracketSpacing: true,
+        explicitTypes: 'always',
+      },
+    },
+  ],
+}

README.md

@@ -12,7 +12,7 @@ yarn install
 
 ## What's included?
 
- :white_check_mark: Cross chain Greeter contract  
+:white_check_mark: Cross chain Greeter contract  
  :white_check_mark: Hardhat configured to Arbitrum's latest testnet  
  :white_check_mark: Solidity interfaces for the Inbox and ArbSys  
- :white_check_mark: `arb-ts` Typescript tooling for cross-layer interactions  
+ :white_check_mark: `arb-ts` Typescript tooling for cross-layer interactions

package.json

@@ -19,13 +19,11 @@
       "packages/token_deposit",
       "packages/token_withdraw",
       "packages/demo-dapp-pet-shop"
-      
     ]
   },
   "scripts": {
     "compile:greeter": "yarn workspace greeter build",
     "compile:eth_deposit": "yarn workspace eth_deposit build",
     "compile:eth_withdraw": "yarn workspace eth_withdraw build"
-
   }
 }

packages/arb-shared-dependencies/package.json

@@ -8,14 +8,13 @@
   "devDependencies": {
     "@nomiclabs/hardhat-ethers": "^2.0.2",
     "arb-ts": "0.0.9",
+    "eslint": "^7.30.0",
     "ethers": "^5.1.2",
-    "hardhat": "^2.2.0"
+    "hardhat": "^2.2.0",
+    "prettier": "^2.3.2"
   },
   "dependencies": {
-    "dotenv": "^8.2.0"
+    "dotenv": "^8.2.0",
+    "eslint-plugin-prettier": "^3.4.0"
   }
 }
-
-
-
-