Skip to content

fix(ci): 修正 E2E Merge Reports 下載路徑導致 main CI 長期紅燈#482

Merged
s123104 merged 1 commit into
mainfrom
fix/ci-e2e-merge-reports-path
Jun 27, 2026
Merged

fix(ci): 修正 E2E Merge Reports 下載路徑導致 main CI 長期紅燈#482
s123104 merged 1 commit into
mainfrom
fix/ci-e2e-merge-reports-path

Conversation

@s123104

@s123104 s123104 commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

背景

main 的「CI」workflow 長期紅燈(#472/#446/#474/#478/#479 連續失敗),根因非程式碼。

根因

E2E Full 兩 shard 全綠並上傳 blob,但 E2E Merge Reports job:

  • download-artifact 下載到 repo 根的 all-blob-reports/
  • merge 指令 pnpm --filter @app/ratewise exec playwright merge-reports ... ./all-blob-reports 的 cwd 為 apps/ratewise/
  • ./all-blob-reports 指向不存在的 apps/ratewise/all-blob-reports → merge 找不到報告 → job 失敗

修法(最小)

download-artifact path 改為 apps/ratewise/all-blob-reports,對齊 merge 指令 cwd 與輸出 apps/ratewise/playwright-report

驗證

合併後 main push 的 E2E Merge Reports 應由 fail 轉 pass。

- E2E Full 兩 shard 全綠,但 merge-reports 因 blob 下載到 repo 根而 cwd 在 apps/ratewise 找不到報告而失敗
- download-artifact path 改為 apps/ratewise/all-blob-reports,對齊 pnpm --filter exec 的套件工作目錄
- 修正後 main push 的 E2E Merge Reports 可正常合併報告,main CI 由長期紅燈轉綠

測試:本機核對 yaml 路徑與 cwd 對齊;修正後以 main push 的 E2E Merge Reports 由 fail 轉 pass 驗證
@s123104 s123104 merged commit 6078b75 into main Jun 27, 2026
@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-merge-reports-path branch June 27, 2026 11:14
@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