refactor: change setup-frontend action to use include_build_step (default false)

- Renamed parameter from skip_build to include_build_step
- Changed default from 'false' to 'false' (building now opt-in)
- Updated all workflow usages to explicitly set include_build_step: 'true' where building is needed
- Removed explicit parameter for workflows that don't need building

As suggested by @DrJKL in PR review
This commit is contained in:
snomiao
2025-10-09 00:57:34 +00:00
parent 4fc2f68b5a
commit 53f44f48fe
5 changed files with 8 additions and 14 deletions

View File

@@ -24,7 +24,7 @@ jobs:
- name: Setup frontend
uses: ./.github/actions/setup-frontend
with:
not_build: 'false'
include_build_step: 'true'
- name: Setup Playwright
uses: ./.github/actions/setup-playwright # Setup Playwright and cache browsers
@@ -65,8 +65,6 @@ jobs:
uses: ./.github/actions/setup-comfyui-server
- name: Setup nodejs, pnpm, reuse built frontend
uses: ./.github/actions/setup-frontend
with:
not_build: 'true'
- name: Setup Playwright
uses: ./.github/actions/setup-playwright
@@ -112,8 +110,6 @@ jobs:
uses: ./.github/actions/setup-comfyui-server
- name: Setup nodejs, pnpm, reuse built frontend
uses: ./.github/actions/setup-frontend
with:
not_build: 'true'
- name: Setup Playwright
uses: ./.github/actions/setup-playwright
@@ -148,8 +144,6 @@ jobs:
# Setup Test Environment, we only need playwright to merge reports
- name: Setup frontend
uses: ./.github/actions/setup-frontend
with:
not_build: 'true'
- name: Setup Playwright
uses: ./.github/actions/setup-playwright