Merge pull request #315 from 10up/fix/build-step

Fix our build step so it includes needed dependencies
10up · Nov 12, 2024 · 5c97303 · 5c97303
2 parents 9e0ebd1 + 1813be6
commit 5c97303
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 41 deletions.
diff --git a/.github/workflows/build-and-tag.yml b/.github/workflows/build-and-tag.yml
diff --git a/.github/workflows/push-deploy.yml b/.github/workflows/push-deploy.yml
@@ -9,12 +9,34 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout code
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
+
+    - name: Setup node version and npm cache
+      uses: actions/setup-node@v4
+      with:
+        node-version-file: '.nvmrc'
+
+    - name: Install Node dependencies
+      run: npm install
+
+    - name: Build plugin
+      run: npm run build
 
     - name: WordPress Plugin Deploy
-      if: "! github.event.release.prerelease"
       id: deploy
       uses: 10up/action-wordpress-plugin-deploy@stable
+      with:
+        generate-zip: true
       env:
         SVN_USERNAME: ${{ secrets.SVN_USERNAME }}
-        SVN_PASSWORD: ${{ secrets.SVN_PASSWORD }}
+        SVN_PASSWORD: ${{ secrets.SVN_PASSWORD }}
+
+    - name: Upload release asset
+      uses: actions/upload-release-asset@v1
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      with:
+        upload_url: ${{ github.event.release.upload_url }}
+        asset_path: ${{ github.workspace }}/${{ github.event.repository.name }}.zip
+        asset_name: ${{ github.event.repository.name }}.zip
+        asset_content_type: application/zip