bug(CI): Only update the shiny submodule when making the dev docs (#1417

)
posit-dev · May 23, 2024 · 8864354 · 8864354
1 parent b1eaaaf
commit 8864354
Showing 1 changed file with 15 additions and 8 deletions.
diff --git a/.github/workflows/build-docs.yaml b/.github/workflows/build-docs.yaml
@@ -45,12 +45,6 @@ jobs:
           cd docs
           make quartodoc
 
-      - name: Build site
-        if: ${{ github.ref == 'refs/heads/main' || github.event_name == 'merge_group' || startsWith(github.head_ref, 'docs')  }}
-        run: |
-          cd docs
-          make site
-
       # =====================================================
       # Shinylive
       # =====================================================
@@ -62,19 +56,32 @@ jobs:
           ref: main
           path: shinylive-repo
 
-      - name: Update shinylive's copy of shiny, htmltools, and other packages
+      - name: Update shinylive's copy of shiny and htmltools
         if: github.ref == 'refs/heads/main'
         run: |
           cd shinylive-repo
           make submodules
-          make submodules-pull
+          make submodules-pull-shiny
+          make submodules-pull-htmltools
 
       - name: Build shinylive
         if: github.ref == 'refs/heads/main'
         run: |
           cd shinylive-repo
           make all
 
+      # TODO-future; Install updated shinylive assets via updated py-shinylive pkg
+
+      # =====================================================
+      # Build site
+      # =====================================================
+
+      - name: Build site
+        if: ${{ github.ref == 'refs/heads/main' || github.event_name == 'merge_group' || startsWith(github.head_ref, 'docs')  }}
+        run: |
+          cd docs
+          make site
+
       # =====================================================
       # Deploy
       # =====================================================