ci: 限制 iOS 内容缓存快照,停止按 PR 提交重复保存 - #648
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe iOS E2E workflow now uses a shared composite action to manage daily Homebrew, ccache, and Metro caches. Cache writes are enabled only for the core matrix leg, and only on qualifying pushes to master. ChangesiOS E2E cache
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Sequence Diagram(s)sequenceDiagram
participant E2EWorkflow as e2e_ios workflow
participant CacheAction as e2e-cache action
participant ActionsCache as GitHub Actions cache
E2EWorkflow->>CacheAction: Pass path, scope, and write flag
CacheAction->>ActionsCache: Restore using scoped daily key and prefix
opt Push to master in e2e-ios with write enabled
CacheAction->>ActionsCache: Save cache
end
Merge Risk: ⚪ Minimal · up to The cache change appears mergeable after the normal iOS E2E checks; no specific failure requiring a pre-merge fix is established. Architecture SummaryArchitecture risk: 🔵 Low · up to The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency. Changed systems: None identified. Architecture concerns Review detailsBefore / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
改动
保持不变
core/native 两个矩阵、全部原测试和 --retries 1、60 分钟上限、模拟器预启动、Bun/Pods/Detox framework 缓存均保持不变。原生 app 的完整输入键、无前缀回退约束、命中后的 JS bundle 刷新没有改变。失败诊断仍保留 7 天。
本次不拆分原生应用构建,也不增加巨型跨任务应用 artifact;需要冷/热缓存耗时对照后再决定。新的共享缓存第一次运行可能冷启动,日快照节省的是写入/存储重复,不承诺每次编译更快。旧缓存没有被删除。
验证
本地 composite YAML、日期 shell 语法、读写 cache key 一致与默认只读检查通过。完整 iOS E2E 由本 PR 运行验证;没有本地 macOS 构建结果,也不把 YAML 检查当作 E2E 通过。未合并或发版。
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit