Typescript 4.5

.circleci/config.yml

@@ -1,44 +1,108 @@
+version: 2.1
+
 dockerhub_auth: &dockerhub_auth
   auth:
     username: $DOCKERHUB_USERNAME
     password: $DOCKERHUB_PASSWORD
 
-# yaml fragment for deploy filters
 deploy_filter: &deploy_filter
   filters:
     branches:
       ignore:
         - /__.*/
 
-version: 2.1
-
-jobs:
-  test_form_backend:
-    docker:
-      - image: eaadtbs/ib-ci-build:3.0
-        <<: *dockerhub_auth
-    working_directory: "~/InfoBase"
+commands:
+  top_level_install:
     steps:
-      - checkout:
-          path: "~/InfoBase"
       - restore_cache:
           keys:
-            - form-backend-dependencies-{{ checksum "form_backend/package-lock.json" }}-v2
             - top-level-dependencies-{{ checksum "package-lock.json" }}-v2
       - run:
           command: |
-            [ -e "form_backend/node_modules" ] || (cd form_backend && npm ci)
+            [ -e "node_modules" ] || npm ci
+      - save_cache:
+          key: top-level-dependencies-{{ checksum "package-lock.json" }}-v2
+          paths:
+            - node_modules
+  form_backend_install:
+    steps:
+      - restore_cache:
+          keys:
+            - form-backend-dependencies-{{ checksum "form_backend/package-lock.json" }}-v2
       - run:
           command: |
-            [ -e "node_modules" ] || npm ci
+            [ -e "form_backend/node_modules" ] || (cd form_backend && npm ci)
       - save_cache:
           key: form-backend-dependencies-{{ checksum "form_backend/package-lock.json" }}-v2
           paths:
             - form_backend/node_modules
+  server_install:
+    steps:
+      - restore_cache:
+          keys:
+            - server-dependencies-{{ checksum "server/package-lock.json" }}-v2
+      - run:
+          command: |
+            [ -e "server/node_modules" ] || (cd server && npm ci)
       - save_cache:
-          key: top-level-dependencies-{{ checksum "package-lock.json" }}-v2
+          key: server-dependencies-{{ checksum "server/package-lock.json" }}-v2
           paths:
-            - node_modules
+            - server/node_modules
+  client_install:
+    steps:
+      # need to bust the client package cache on patch changes in addition to package-lock changes
+      - run: cksum ./client/patches/* > client-patch-checksums
+      - restore_cache:
+          keys:
+            - client-dependencies-{{ checksum "client/package-lock.json" }}-{{ checksum "client-patch-checksums" }}-v2
+      - run:
+          command: |
+            [ -e "client/node_modules" ] || (cd client && npm ci)
+      - save_cache:
+          key: client-dependencies-{{ checksum "client/package-lock.json" }}-{{ checksum "client-patch-checksums" }}-v2
+          paths:
+            - client/node_modules
+            - client/.cache/Cypress # path set by CYPRESS_CACHE_FOLDER env var
+
+jobs:
+  lint:
+    docker:
+      - image: eaadtbs/ib-ci-build:3.0
+        <<: *dockerhub_auth
+    working_directory: "~/InfoBase"
+    steps:
+      - checkout:
+          path: "~/InfoBase"
+
+      - top_level_install
+      - form_backend_install
+      - server_install
+      - client_install
+
+      - restore_cache:
+          keys:
+            - repo-wide-lint-cache-{{ .Branch }}
+            - repo-wide-lint-cache-master
+
+      - run:
+          command: npm run eslint
+
+      - save_cache:
+          key: repo-wide-lint-cache-{{ .Branch }}
+          paths:
+            - .eslintcache
+
+  test_form_backend:
+    docker:
+      - image: eaadtbs/ib-ci-build:3.0
+        <<: *dockerhub_auth
+    working_directory: "~/InfoBase"
+    steps:
+      - checkout:
+          path: "~/InfoBase"
+
+      - top_level_install
+      - form_backend_install
 
       - restore_cache:
           keys:
@@ -90,49 +154,15 @@ jobs:
       - image: eaadtbs/ib-ci-build:3.0
         <<: *dockerhub_auth
     working_directory: "~/InfoBase"
-    environment:
-      CYPRESS_CACHE_FOLDER: "~/InfoBase/client/.cache/Cypress"
     resource_class: medium
     steps:
       - checkout:
           path: "~/InfoBase"
       - run: ./scripts/ci_scripts/create_envs.sh
 
-      - restore_cache:
-          keys:
-            - top-level-dependencies-{{ checksum "package-lock.json" }}-v4
-      - run:
-          command: |
-            [ -e "node_modules" ] ||  npm ci
-      - save_cache:
-          key: top-level-dependencies-{{ checksum "package-lock.json" }}-v4
-          paths:
-            - node_modules
-
-      # need to bust the client package cache on patch changes in addition to package-lock changes
-      - run: cksum ./client/patches/* > client-patch-checksums
-      - restore_cache:
-          keys:
-            - client-dependencies-{{ checksum "client/package-lock.json" }}-{{ checksum "client-patch-checksums" }}-v2
-      - run:
-          command: |
-            [ -e "client/node_modules" ] || (cd client && npm ci)
-      - save_cache:
-          key: client-dependencies-{{ checksum "client/package-lock.json" }}-{{ checksum "client-patch-checksums" }}-v2
-          paths:
-            - client/node_modules
-            - client/.cache/Cypress
-
-      - restore_cache:
-          keys:
-            - server-dependencies-{{ checksum "server/package-lock.json" }}-v2
-      - run:
-          command: |
-            [ -e "server/node_modules" ] || (cd server && npm ci)
-      - save_cache:
-          key: server-dependencies-{{ checksum "server/package-lock.json" }}-v2
-          paths:
-            - server/node_modules
+      - top_level_install
+      - client_install
+      - server_install
 
       # transpiling the server slightly out of place for this job, not needed till the server deploy step, but
       # the deploy job image isn't guaranteed to have the necessary environment for this
@@ -272,8 +302,6 @@ jobs:
       - image: eaadtbs/ib-ci-cypress:4.0
         <<: *dockerhub_auth
     working_directory: "~/InfoBase"
-    environment:
-      CYPRESS_CACHE_FOLDER: "~/InfoBase/client/.cache/Cypress"
     parameters:
       batch-count:
         type: integer
@@ -380,6 +408,10 @@ jobs:
       - run: if [[ $CIRCLE_BRANCH = master ]]; then ./scripts/ci_scripts/store_bundle_stats.sh; fi
 
 workflows:
+  lint:
+    jobs:
+      - lint
+
   test_form_backend:
     jobs:
       - test_form_backend

.eslintignore

@@ -0,0 +1,7 @@
+*/.coverage
+*/.cache
+*/transpiled_build
+client/build
+client/build_prod
+client/src/**/*.scss.d.ts
+client/scripts/typescript_progress/output

.eslintrc.json

@@ -8,7 +8,10 @@
     "plugin:import/errors"
   ],
   "rules": {
-    "no-unused-vars": ["warn", { "args": "none" }],
+    "no-unused-vars": [
+      "warn",
+      { "ignoreRestSiblings": true, "argsIgnorePattern": "^_.+" }
+    ],
     "no-use-before-define": ["error", { "functions": true, "classes": true }],
     "no-throw-literal": "error",
     "lodash/no-double-unwrap": ["error"],

.gitignore

@@ -20,4 +20,5 @@ npm-debug*
 node_modules/
 .local_mongo
 .coverage
-.cache
+.cache
+.eslintcache

.husky/pre-push

@@ -22,6 +22,5 @@ if [[ ($local_branch_name =~ ^__ || $local_branch_name =~ ^archived__ ) && $remo
   exit 1
 fi
 
-
-# run prettier check
-npm run prettier_check .
+# Prevent pushing if prettier is unhappy
+npm run prettier

client/.eslintrc.json

@@ -47,8 +47,11 @@
       ],
       "rules": {
         "no-prototype-builtins": "off", //common TS pattern
-        "@typescript-eslint/no-unused-vars": "off", // Lot of false positives on this rule. Better to use the exprimental
-        "@typescript-eslint/no-unused-vars-experimental": "warn",
+        "no-unused-vars": "off",
+        "@typescript-eslint/no-unused-vars": [
+          "warn",
+          { "ignoreRestSiblings": true, "argsIgnorePattern": "^_.+" }
+        ],
         "@typescript-eslint/no-redeclare": ["error"],
         "@typescript-eslint/explicit-module-boundary-types": "off", // TODO might want to turn this on later, but it will take a while to fix all the warnings
         "@typescript-eslint/ban-types": [