Merge pull request #14 from broadinstitute/fix-bundling

Added bundling of js code before building docker image

Author: pgm

.github/workflows/build-docker.yaml

@@ -1,8 +1,6 @@
 name: Build Taiga image
 
-on: 
-    push:
-        branches: main
+on: push
 
 env:
   IMAGE_TAG: us.gcr.io/cds-docker-containers/taiga:ga-build-${{ github.run_number }}
@@ -18,6 +16,8 @@ jobs:
       - uses: actions/checkout@v2
       - name: write SHA to file
         run: "bash write_version.sh ${{ github.sha }}"
+      - name: bundle JS
+        run: "bash bundle_js.sh"
       - name: Login to GCR
         uses: docker/login-action@v2
         with:
@@ -31,6 +31,8 @@ jobs:
       - name: Push Docker image
         run: docker push ${{ env.IMAGE_TAG }}
       - name: Tag as latest
+        if: github.ref == 'refs/heads/main'
         run: docker tag ${{ env.IMAGE_TAG }} ${{ env.FINAL_IMAGE_TAG }}
       - name: Push Docker image
+        if: github.ref == 'refs/heads/main'
         run: docker push ${{ env.FINAL_IMAGE_TAG }}

bundle_js.sh

@@ -0,0 +1,4 @@
+set -ex
+cd react_frontend
+yarn install
+./node_modules/.bin/webpack --mode=production

install_prereqs.sh

@@ -1,5 +1,4 @@
 set -ex
-pip install -r dev-requirements.txt
-python setup.py develop
 pre-commit install
+poetry install
 yarn install --cwd react_frontend