Skip to content

Commit 8095a4b

Browse files
committed
ci: optimize build pipeline to eliminate redundant builds
- validate now uses test:all directly (build already done) - Added test:ci script for explicit CI usage - ci.yml now builds UI deps and uses build:all - dev-release.yml separates build and validate steps - Reduces build count from 3-4 to 1 per release cycle
1 parent 70d54b9 commit 8095a4b

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,12 @@ jobs:
2323
node-version: '22'
2424

2525
- name: Install dependencies
26-
run: bun install --frozen-lockfile
26+
run: |
27+
bun install --frozen-lockfile
28+
cd ui && bun install --frozen-lockfile
2729
2830
- name: Build package
29-
run: bun run build
31+
run: bun run build:all
3032

3133
- name: Validate (typecheck + lint + tests)
3234
run: bun run validate

.github/workflows/dev-release.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,11 @@ jobs:
3636
bun install --frozen-lockfile
3737
cd ui && bun install --frozen-lockfile
3838
39-
- name: Build and validate
40-
run: |
41-
bun run build:all
42-
bun run validate
39+
- name: Build
40+
run: bun run build:all
41+
42+
- name: Validate (typecheck + lint + tests)
43+
run: bun run validate
4344

4445
- name: Bump dev version
4546
id: bump

package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,10 @@
6262
"lint:fix": "eslint src/ --fix",
6363
"format": "prettier --write src/",
6464
"format:check": "prettier --check src/",
65-
"validate": "bun run typecheck && bun run lint:fix && bun run format:check && bun run test",
65+
"validate": "bun run typecheck && bun run lint:fix && bun run format:check && bun run test:all",
6666
"verify:bundle": "node scripts/verify-bundle.js",
6767
"test": "bun run build && bun run test:all",
68+
"test:ci": "bun run test:all",
6869
"test:all": "bun test",
6970
"test:unit": "bun test tests/unit/",
7071
"test:npm": "bun test tests/npm/",

0 commit comments

Comments
 (0)