Repository navigation
ci(ios): cache agent-device's XCUITest runner - #1185
Conversation
There was a problem hiding this comment.
🔵 Needs a closer look
The cache may retain session data and become stale when package contents change without a version change.
0 open findings
What changed in this PR
Adds GitHub Actions caching for the agent-device iOS XCUITest runner to reduce CI setup time.
Changes:
- Restores and saves the runner cache around iOS tests.
- Keys caching by platform, architecture, package version, and Xcode version.
- Attempts to exclude session-specific files.
| File | Summary |
|---|---|
.github/workflows/ios.yml |
Adds runner caching; cache exclusion and key composition require changes. |
🧠 Review effort: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
🟢 e2e: 13 passedAll 13 tests in 11 files
e2e 0.21.0 · 4m 25s · ios · run artifacts |
|
@krozniata Could you change the first cache path from Products to Products/*? Caching the whole directory causes tar to include the session files despite the exclusion pattern. |
ArekChr
left a comment
There was a problem hiding this comment.
LGTM, Both iOS jobs passed all 13 E2E tests, including a successful cache restore
Summary
Test Plan
What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility
Checklist
README.md