Skip to content

Commit 5e3aa18

Browse files
committed
improvement(ci): parallelize Docker builds with tests and remove duplicate turbo install
1 parent eabf2de commit 5e3aa18

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
# Build AMD64 images and push to ECR immediately (+ GHCR for main)
4949
build-amd64:
5050
name: Build AMD64
51-
needs: [test-build, detect-version]
51+
needs: [detect-version]
5252
if: github.event_name == 'push' && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/staging' || github.ref == 'refs/heads/dev')
5353
runs-on: blacksmith-8vcpu-ubuntu-2404
5454
permissions:
@@ -150,7 +150,7 @@ jobs:
150150
# Build ARM64 images for GHCR (main branch only, runs in parallel)
151151
build-ghcr-arm64:
152152
name: Build ARM64 (GHCR Only)
153-
needs: [test-build, detect-version]
153+
needs: [detect-version]
154154
runs-on: blacksmith-8vcpu-ubuntu-2404-arm
155155
if: github.event_name == 'push' && github.ref == 'refs/heads/main'
156156
permissions:

docker/app.Dockerfile

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,10 @@ COPY packages/testing/package.json ./packages/testing/package.json
2525
COPY packages/logger/package.json ./packages/logger/package.json
2626
COPY packages/tsconfig/package.json ./packages/tsconfig/package.json
2727

28-
# Install turbo globally, then dependencies, then rebuild isolated-vm for Node.js
28+
# Install dependencies, then rebuild isolated-vm for Node.js
2929
# Use --linker=hoisted for flat node_modules layout (required for Docker multi-stage builds)
3030
RUN --mount=type=cache,id=bun-cache,target=/root/.bun/install/cache \
3131
--mount=type=cache,id=npm-cache,target=/root/.npm \
32-
bun install -g turbo && \
3332
HUSKY=0 bun install --omit=dev --ignore-scripts --linker=hoisted && \
3433
cd node_modules/isolated-vm && npx node-gyp rebuild --release
3534

0 commit comments

Comments
 (0)