Skip to content

Commit d9352b6

Browse files
ryan-williamsclaude
andcommitted
Build UI before running e2e tests in CI
The e2e tests need the built UI files to serve. Previously we only created an empty `ui/dist` directory which caused 404s for all pages. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 9c4b34d commit d9352b6

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

.github/workflows/ci.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -92,13 +92,7 @@ jobs:
9292
- name: Set up Python
9393
run: uv python install 3.12
9494

95-
- name: Create empty ui/dist for editable install
96-
run: mkdir -p ui/dist
97-
98-
- name: Install Python dependencies
99-
run: uv sync
100-
101-
# Node.js setup for Playwright
95+
# Node.js setup for UI build and Playwright
10296
- uses: pnpm/action-setup@v4
10397
with:
10498
version: 9
@@ -112,6 +106,12 @@ jobs:
112106
- name: Install UI dependencies
113107
run: cd ui && pnpm install --frozen-lockfile
114108

109+
- name: Build UI
110+
run: cd ui && pnpm build
111+
112+
- name: Install Python dependencies
113+
run: uv sync
114+
115115
- name: Install Playwright browsers
116116
run: cd ui && pnpm exec playwright install chromium --with-deps
117117

0 commit comments

Comments
 (0)