Merge pull request #363 from aeisenberg/lint-fixes

Fixes to how we lint automatically

Author: jcreedcmu

extensions/ql-vscode/.eslintrc.js

@@ -26,10 +26,7 @@ module.exports = {
     "@typescript-eslint/no-explicit-any": "off",
     "prefer-const": ["warn", {"destructuring": "all"}],
     "indent": "off",
-    "@typescript-eslint/indent": ["error", 2, {
-      "SwitchCase": 1,
-      "FunctionDeclaration": { "body": 1, "parameters": 1 }
-    }],
+    "@typescript-eslint/indent": "off",
     "@typescript-eslint/no-throw-literal": "error",
     "no-useless-escape": 0
   },

extensions/ql-vscode/package.json

@@ -480,10 +480,10 @@
     "preintegration": "rm -rf ./out/vscode-tests && gulp",
     "integration": "node ./out/vscode-tests/run-integration-tests.js",
     "update-vscode": "node ./node_modules/vscode/bin/install",
-    "postinstall": "node ./node_modules/vscode/bin/install",
+    "postinstall": "npm rebuild && node ./node_modules/vscode/bin/install",
     "format": "tsfmt -r",
     "lint": "eslint src test --ext .ts,.tsx",
-    "lint-staged": "lint-staged"
+    "format-staged": "lint-staged"
   },
   "dependencies": {
     "child-process-promise": "^2.2.1",
@@ -567,15 +567,15 @@
   },
   "husky": {
     "hooks": {
-      "pre-commit": "npm run lint-staged"
+      "pre-commit": "npm run format-staged",
+      "pre-push": "npm run lint"
     }
   },
   "lint-staged": {
     "./**/*.{json,css,scss,md}": [
       "prettier --write"
     ],
     "./**/*.{ts,tsx}": [
-      "eslint --fix --debug",
       "tsfmt -r"
     ]
   }