Better ci (#127)

* Use `npm ci` and build app on PRs

* deploy on main

* Use same CI env var setting for PR pipeline and deployment

---------

Co-authored-by: Benjamin Karran <[email protected]>

Author: ebekebe

.github/workflows/github-pages.yml

@@ -1,9 +1,9 @@
 name: github pages release
 
-on: workflow_dispatch
-#   push:
-#     branches:
-#       - main
+on:
+  push:
+    branches:
+      - main
 #     paths:
 #       - '.storybook'
 #       - 'public'
@@ -28,7 +28,7 @@ jobs:
           CI: false
         run: |
           ls
-          npm install
+          npm ci
           npm run build
 
       - name: Build storybook

.github/workflows/sample-app-web.yml

@@ -37,7 +37,10 @@ jobs:
 
     # Site Testing steps
     - name: Install dependencies
-      run: npm install
+      run: npm ci
+
+    - name: Build
+      run: CI=false npm run build
 
     - name: Run Unit Tests and generate coverage report
       run: npm run test.coverage

Dockerfile

@@ -8,4 +8,4 @@ ENV PATH="$HOME/sc-4.8.2-linux/bin:$PATH"
 # web app
 WORKDIR /sample-app-web
 COPY . .
-RUN npm install
+RUN npm ci