ci: simplify Mac workflow by removing Node.js dependency

Remove the dependency on Node.js CI job for Mac and iOS builds, and
always build web artifacts locally in the Mac workflow. This simplifies
the CI pipeline and removes the artifact download step that could fail.

Changes:
- Remove `node` dependency from `mac` and `ios` jobs in ci.yml
- Replace artifact download with direct web build in mac.yml
- Simplify the build process by removing conditional logic

This extracts the CI workflow improvements from PR #318.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Peter Steinberger 2025-07-12 19:07:55 +02:00
parent 1052354394
commit ac55f9685c
2 changed files with 9 additions and 20 deletions

View file

@ -52,7 +52,7 @@ jobs:
mac: mac:
name: Mac CI name: Mac CI
needs: [changes, node] needs: [changes]
if: | if: |
always() && always() &&
!contains(needs.*.result, 'failure') && !contains(needs.*.result, 'failure') &&
@ -62,7 +62,7 @@ jobs:
ios: ios:
name: iOS CI name: iOS CI
needs: [changes, node] needs: [changes]
if: | if: |
always() && always() &&
!contains(needs.*.result, 'failure') && !contains(needs.*.result, 'failure') &&

View file

@ -141,25 +141,14 @@ jobs:
fi fi
done done
- name: Download web build artifacts - name: Build web artifacts
uses: actions/download-artifact@v4
continue-on-error: true
with:
name: web-build-${{ github.sha }}
path: web/
- name: Build web artifacts if missing
run: | run: |
if [ ! -d "web/dist" ] || [ ! -d "web/public/bundle" ]; then echo "Building web artifacts locally..."
echo "Web build artifacts not found, building locally..." cd web
cd web # Skip custom Node.js build in CI to avoid timeout
# Skip custom Node.js build in CI to avoid timeout export CI=true
export CI=true pnpm run build
pnpm run build echo "Web artifacts built successfully"
echo "Web artifacts built successfully"
else
echo "Web build artifacts found, skipping local build"
fi
- name: Resolve Dependencies (once) - name: Resolve Dependencies (once)
run: | run: |