Skip to content

ci(release): keep iOS dSYM artifacts beside the IPA - #645

Merged
PiscesXD merged 1 commit into
mainfrom
ci/keep-iOS-dSYM-artifacts-beside-the-IPA
Oct 24, 2026
Merged

PiscesXD merged 1 commit into
mainfrom
ci/keep-iOS-dSYM-artifacts-beside-the-IPA

Conversation

@lowrt

@lowrt lowrt commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

這個 PR 做了什麼

將 iOS dSYM ZIP 改存於 IPA 所在的 workspace,避免 upload-artifact 因同時收集 workspace 與 $RUNNER_TEMP 的檔案而在下載結果中保留 _temp 目錄。

先前 Publish job 使用 artifacts/* 上傳 GitHub Release assets,會誤將 artifacts/_temp 目錄交給 gh release create,因而出現:

text
read artifacts/_temp: is a directory
message: "Bad Content-Length"

相關 issue

沒有對應的 issue。

怎麼驗

tool/check/workflows.sh 通過
tool/check/workflow_security.sh 通過
尚未實際執行完整 Release workflow;合併後由下一次 snapshot 發布確認 GitHub Release assets 正常上傳

檢查清單

  • tool/check/commits.sh origin/main..HEAD 通過
    —— commit 訊息就是更新日誌,格式見 commit.md
  • 一個 commit 一件事(這條 gate 驗不了,靠自己和 review)
  • mise exec -- flutter analyze 與 mise exec -- flutter test 通過
  • 新的使用者可見字串都走 AppLocalizations,沒有寫死
  • CI 全部通過(gh pr checks:每一項都是 pass 或 skipped)—— 沒過不能合併,
    見 pr.md
  • 有 UI 變更的話:用的是 AppSpacing / AppRadius / AppMotion,
    深色模式看過,文字對比度可接受

@codecov

codecov Bot commented Oct 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.49%. Comparing base (ce4b24b) to head (be7a5bb).

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@YinCheng0106 YinCheng0106 added the ios Issues or pull requests related to the iOS platform. label Oct 23, 2026
@YinCheng0106
YinCheng0106 requested a review from a team October 24, 2026 04:12
@PiscesXD
PiscesXD merged commit 5b3a103 into main Oct 24, 2026
8 checks passed
@PiscesXD
PiscesXD deleted the ci/keep-iOS-dSYM-artifacts-beside-the-IPA branch October 24, 2026 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ios Issues or pull requests related to the iOS platform. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants