fix(ci): remove build-check job from CI workflow (#66)
* ci: remove build-check job from CI workflow The cross-platform build verification is redundant in CI since the build and release pipeline (build.yml) already performs full builds on every release. This reduces CI runner time and costs. * ci: restrict build-check to Linux PRs and pin R2 action to v1.4 - Build-check now only runs on pull requests (skipped on main merge) with only Linux/amd64 enabled; macOS and Windows are commented out since the Build & Publish workflow handles all platforms on release. - Pin ryand56/r2-upload-action from floating v1 tag to v1.4 to fix the upload failure seen in the v1.1.0 release build.
This commit is contained in:
@@ -176,7 +176,7 @@ jobs:
|
|||||||
artifacts/release/yellowjacket-windows-amd64.exe
|
artifacts/release/yellowjacket-windows-amd64.exe
|
||||||
|
|
||||||
- name: Upload to R2
|
- name: Upload to R2
|
||||||
uses: ryand56/r2-upload-action@v1
|
uses: ryand56/r2-upload-action@v1.4
|
||||||
with:
|
with:
|
||||||
r2-account-id: ${{ secrets.R2_ACCOUNT_ID }}
|
r2-account-id: ${{ secrets.R2_ACCOUNT_ID }}
|
||||||
r2-access-key-id: ${{ secrets.R2_ACCESS_KEY_ID }}
|
r2-access-key-id: ${{ secrets.R2_ACCESS_KEY_ID }}
|
||||||
|
|||||||
+10
-10
@@ -141,8 +141,6 @@ jobs:
|
|||||||
|
|
||||||
# ──────────────────────────────────────────────
|
# ──────────────────────────────────────────────
|
||||||
# Frontend: type-check
|
# Frontend: type-check
|
||||||
# (full build is verified by the build-check job
|
|
||||||
# via Wails which handles path resolution)
|
|
||||||
# ──────────────────────────────────────────────
|
# ──────────────────────────────────────────────
|
||||||
frontend:
|
frontend:
|
||||||
name: Frontend Type Check
|
name: Frontend Type Check
|
||||||
@@ -170,11 +168,13 @@ jobs:
|
|||||||
run: pnpm exec tsc --noEmit
|
run: pnpm exec tsc --noEmit
|
||||||
|
|
||||||
# ──────────────────────────────────────────────
|
# ──────────────────────────────────────────────
|
||||||
# Build verification: ensure it compiles on all
|
# Build verification: ensure it compiles
|
||||||
# target platforms (no artifacts uploaded)
|
# (only runs on PRs; release builds are handled
|
||||||
|
# by the Build & Publish workflow)
|
||||||
# ──────────────────────────────────────────────
|
# ──────────────────────────────────────────────
|
||||||
build-check:
|
build-check:
|
||||||
name: Build Check (${{ matrix.platform }})
|
name: Build Check (${{ matrix.platform }})
|
||||||
|
if: github.event_name == 'pull_request'
|
||||||
needs: [go-checks, go-test, codegen-check, frontend]
|
needs: [go-checks, go-test, codegen-check, frontend]
|
||||||
strategy:
|
strategy:
|
||||||
fail-fast: false
|
fail-fast: false
|
||||||
@@ -184,12 +184,12 @@ jobs:
|
|||||||
os: ubuntu-latest
|
os: ubuntu-latest
|
||||||
build-name: yellowjacket
|
build-name: yellowjacket
|
||||||
apt-deps: libasound2-dev
|
apt-deps: libasound2-dev
|
||||||
- platform: darwin/universal
|
# - platform: darwin/universal
|
||||||
os: macos-latest
|
# os: macos-latest
|
||||||
build-name: yellowjacket
|
# build-name: yellowjacket
|
||||||
- platform: windows/amd64
|
# - platform: windows/amd64
|
||||||
os: windows-latest
|
# os: windows-latest
|
||||||
build-name: yellowjacket.exe
|
# build-name: yellowjacket.exe
|
||||||
runs-on: ${{ matrix.os }}
|
runs-on: ${{ matrix.os }}
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v6
|
- uses: actions/checkout@v6
|
||||||
|
|||||||
Reference in New Issue
Block a user