Skip to content

fix(ci): 修正 E2E 多 reporter CLI 語法讓 blob 報告產出#484

Merged
s123104 merged 1 commit into
mainfrom
fix/ci-e2e-blob-reporter-comma
Jun 27, 2026
Merged

fix(ci): 修正 E2E 多 reporter CLI 語法讓 blob 報告產出#484
s123104 merged 1 commit into
mainfrom
fix/ci-e2e-blob-reporter-comma

Conversation

@s123104

@s123104 s123104 commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

根因(接續 #482

#482 修了 merge 下載路徑,但 main CI 仍紅。深入發現真正根因:

E2E 測試步驟用三個分開的 --reporter= flag--reporter=blob --reporter=github --reporter=list)。Playwright CLI 的 --reporter 多 reporter 必須逗號分隔;重複 flag 只有最後的 list 生效,blob 從未產出apps/ratewise/blob-report 為空 → upload 無檔(僅 warn)→ download 抓不到 → merge-reports 報「Directory does not exist」失敗。

修法

依 Playwright 官方文檔改為單一逗號語法 --reporter=blob,github,list,blob/github/list 三者同時生效。

依據

context7 / Playwright 官方 sharding 文檔:--reporter 支援逗號分隔多 reporter。

驗證

合併後 main push 的 E2E Full 將產出 blob-report-1/2 artifact,E2E Merge Reports 由 fail 轉 pass。

- 重複的 --reporter= flag 只有最後 list 生效,blob 從未產出,blob-report 為空
- 連帶 merge-reports 找不到報告而失敗,是 main CI 長期紅燈的真正根因
- 依 Playwright 官方文檔改為單一逗號語法 --reporter=blob,github,list,三者同時生效
- 與 PR482 路徑修正合計,E2E Merge Reports 應由 fail 轉 pass

測試:context7 核對 Playwright reporter 逗號語法;以 main push 的 E2E Merge Reports 轉 pass 驗證
@s123104 s123104 merged commit 7254028 into main Jun 27, 2026
5 checks passed
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@s123104 s123104 deleted the fix/ci-e2e-blob-reporter-comma branch June 27, 2026 11:40
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant