chore: add linting configuration based on other repos

.eslint-tsconfig

@@ -0,0 +1,4 @@
+{
+  "extends": "./tsconfig.json",
+  "include": ["jest.config.js", ".eslintrc.js", "tests", "scenario", "deploy"]
+}

.eslintignore

@@ -2,3 +2,7 @@ node_modules
 artifacts
 cache
 coverage
+dist
+# ignoring scenario dir because it contains deprecated saddle code and will be removed
+scenario
+typechain

.eslintrc.js

@@ -7,18 +7,32 @@ module.exports = {
   },
   plugins: ["@typescript-eslint"],
   extends: [
-    "standard",
-    "plugin:prettier/recommended",
-    "plugin:node/recommended",
+    "eslint:recommended",
+    "plugin:@typescript-eslint/eslint-recommended",
+    "plugin:@typescript-eslint/recommended",
+    "prettier",
   ],
   parser: "@typescript-eslint/parser",
   parserOptions: {
     ecmaVersion: 12,
+    project: ".eslint-tsconfig",
   },
+  root: true,
   rules: {
-    "node/no-unsupported-features/es-syntax": [
+    "@typescript-eslint/no-floating-promises": [
       "error",
-      { ignores: ["modules"] },
+      {
+        ignoreIIFE: true,
+        ignoreVoid: true,
+      },
+    ],
+    "@typescript-eslint/no-inferrable-types": "off",
+    "@typescript-eslint/no-unused-vars": [
+      "error",
+      {
+        argsIgnorePattern: "_",
+        varsIgnorePattern: "_",
+      },
     ],
   },
 };

.github/workflows/ci.yaml

@@ -23,7 +23,7 @@ jobs:
         # Hack to get around failing "ethereumjs-abi The remote archive doesn't match the expected checksum" error
         run: YARN_CHECKSUM_BEHAVIOR=update yarn
 
-      - name: Lint solidity
+      - name: Check linting of solidity and typescript
         run: yarn lint
 
   build:

.prettierignore

@@ -3,3 +3,7 @@ artifacts
 cache
 coverage*
 gasReporterOutput.json
+dist
+scenario
+typechain
+contracts/oracle

.prettierrc

@@ -1 +1,16 @@
-{}
+arrowParens: avoid
+bracketSpacing: true
+endOfLine: auto
+importOrder: ["<THIRD_PARTY_MODULES>", "^[./]"]
+importOrderParserPlugins: ["typescript"]
+importOrderSeparation: true
+importOrderSortSpecifiers: true
+printWidth: 120
+singleQuote: false
+tabWidth: 2
+trailingComma: all
+
+overrides:
+  - files: "*.sol"
+    options:
+      tabWidth: 4

package.json

@@ -16,9 +16,13 @@
     "hardhat:test": "hardhat test",
     "hardhat:compile": "hardhat compile",
     "hardhat:coverage": "hardhat coverage",
-    "lint": "yarn lint:sol",
+    "lint": "yarn lint:sol && yarn lint:ts && yarn prettier:check",
     "lint:sol": "solhint \"contracts/**/*.sol\"",
     "lint:sol:fix": "prettier --write \"contracts/**/*.sol\"",
+    "lint:ts": "eslint --ext .js,.ts .",
+    "lint:ts:fix": "eslint --fix --ext .js,.ts .",
+    "prettier": "prettier --write \"**/*.{js,json,md,sol,ts,yaml,yml}\"",
+    "prettier:check": "prettier --check \"**/*.{js,json,md,sol,ts,yaml,yml}\"",
     "build": "rm -rf dist && tsc --declaration && hardhat compile && cp -r ./{package.json,yarn.lock,artifacts,networks,contracts,deployments,copy_contracts.sh} dist/",
     "publish:dist": "yarn build && yarn publish dist -f --access public",
     "postinstall": "./copy_contracts.sh",
@@ -45,20 +49,20 @@
     "@nomicfoundation/hardhat-toolbox": "^2.0.0",
     "@nomiclabs/hardhat-ethers": "npm:hardhat-deploy-ethers",
     "@nomiclabs/hardhat-etherscan": "^3.0.3",
+    "@trivago/prettier-plugin-sort-imports": "^3.4.0",
     "@typechain/ethers-v5": "^7.2.0",
     "@typechain/hardhat": "^2.3.1",
     "@types/chai": "^4.3.1",
     "@types/mocha": "^9.1.1",
     "@types/node": "^12.20.50",
-    "@typescript-eslint/eslint-plugin": "^4.33.0",
-    "@typescript-eslint/parser": "^4.33.0",
+    "@typescript-eslint/eslint-plugin": "^5.27.1",
+    "@typescript-eslint/parser": "^5.27.1",
     "@venusprotocol/oracle": "^1.2.0",
     "bignumber.js": "9.0.0",
     "chai": "^4.3.6",
     "dotenv": "^10.0.0",
     "eslint": "^7.32.0",
     "eslint-config-prettier": "^8.5.0",
-    "eslint-config-standard": "^16.0.3",
     "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^3.4.1",