Bundling does not require type-checking

Bundling the frontend does not necessarily require type checking,
nor should type checking imply bundling.

TSC is also fairly slow. Separate the two and invoke tsc separately in
CI.

.github/workflows/ci.yml

@@ -28,7 +28,22 @@ jobs:
       - uses: Swatinem/rust-cache@v1.3.0
       - run: cargo clippy --workspace --all-targets -- -D warnings
 
-  build_frontend:
+  check_frontend:
+    defaults:
+      run:
+        working-directory: frontend
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
+        with:
+          cache: 'yarn'
+          cache-dependency-path: frontend/yarn.lock
+      - run: yarn install
+      - run: yarn run eslint
+      - run: yarn run tsc
+
+  build_and_test_frontend:
     strategy:
       matrix:
         app: [ maker, taker ]
@@ -43,7 +58,6 @@ jobs:
           cache: 'yarn'
           cache-dependency-path: frontend/yarn.lock
       - run: yarn install
-      - run: yarn run eslint
       - run: APP=${{ matrix.app }} yarn test
       - run: APP=${{ matrix.app }} yarn build
 

frontend/package.json

@@ -3,7 +3,7 @@
   "version": "0.0.0",
   "scripts": {
     "dev": "vite",
-    "build": "tsc && vite build",
+    "build": "vite build",
     "serve": "vite preview",
     "test": "vite-jest"
   },