Merge pull request #51 from CMU-313/frontend-deployment

Removing npm install and only installing frontend after setup
CMU-313 · Oct 22, 2024 · 4e88c8f · 4e88c8f
2 parents 2d42ac6 + b337387
commit 4e88c8f
Showing 1 changed file with 8 additions and 12 deletions.
diff --git a/.github/workflows/azure-deploy-f24.yml b/.github/workflows/azure-deploy-f24.yml
@@ -32,18 +32,6 @@ jobs:
         with:
           node-version: '20.17.0'
 
-      - name: Clone frontend repo as a subdirectory
-        run: |
-          git clone https://github.com/CMU-313/nodebb-frontend-f24-team-software-stars.git
-      
-      - name: Install NodeBB dependencies
-        run: |
-          npm install
-      
-      - name: Install frontend repo as a dependency
-        run: |
-          npm install ./nodebb-frontend-f24-team-software-stars
-
       - name: Set up NodeBB
         run: |
           ./nodebb setup '{"url":"https://nodebb-software-stars.azurewebsites.net:443",
@@ -55,6 +43,14 @@ jobs:
             "redis:host": "${{ secrets.REDIS_HOST }}",
             "redis:port": "6379",
             "redis:password": "${{ secrets.REDIS_PASSWORD }}" }'
+
+      - name: Clone frontend repo as a subdirectory
+        run: |
+          git clone https://github.com/CMU-313/nodebb-frontend-f24-team-software-stars.git
+      
+      - name: Install frontend repo as a dependency
+        run: |
+          npm install ./nodebb-frontend-f24-team-software-stars
           
       - name: 'Deploy to Azure Web App'
         id: deploy-to-webapp