Skip to content

ci(ios): stop CocoaPods job restoring the SwiftPM ccache - #1200

Open
krozniata wants to merge 1 commit into
mainfrom
fix/ccache-bug
Open

krozniata wants to merge 1 commit into
mainfrom
fix/ccache-bug

Conversation

@krozniata

Copy link
Copy Markdown
Member

Summary

Fixes the iOS CI ccache restore key. ccache-ios- also matched ccache-ios-spm-…, so the CocoaPods job could restore the SwiftPM job's ccache and get 0 hits

The keys are now ccache-ios-pods-… and ccache-ios-spm-…, so neither matches the other.

Test Plan

What's required for testing (prerequisites)?

What are the steps to reproduce (after prerequisites)?

Compatibility

OS Implemented
iOS ✅❌
Android ✅❌

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)

Copilot AI lite review requested due to automatic review settings October 26, 2026 11:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The cache key changes address the cross-variant restore issue, with no unresolved review comments.

No open issues 🎉

What changed in this PR

This pull request isolates CocoaPods and SwiftPM iOS CI ccache namespaces so each job restores only its own cache.

Changes:

  • Adds a CACHE_VARIANT based on the spm workflow input.
  • Uses variant-specific ccache and restore-key prefixes.
File Description
.github/​workflows/​ios.yml Separates ccache namespaces for CocoaPods and SwiftPM iOS builds.

Review effort: Lite

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

🟢 e2e: 21 passed

All 21 tests in 12 files
Test Time
🟢 tests/basic-pager.e2e.ts · 3 passed 1m 42s
🟢 basic pager › in LTR › pages horizontally 35.7s
🟢 basic pager › in LTR › pages vertically 24.3s
🟢 basic pager › in RTL › pages horizontally with the reversed gesture 41.8s
🟢 tests/issues/issue-1050-back-gesture.e2e.ts · 8 passed 2m 7s
🟢 back gesture over the pager › in LTR › a back swipe on the first page pops the screen 10.0s
🟢 back gesture over the pager › in LTR › a back swipe on a later page goes to the previous page 10.3s
🟢 back gesture over the pager › in LTR › a forward swipe on the first page still goes to the next page 9.4s
🟢 back gesture over the pager › in RTL › a back swipe on the first page pops the screen 22.4s
🟢 back gesture over the pager › in RTL › a back swipe on a later page goes to the previous page 22.1s
🟢 back gesture over the pager › in RTL › a forward swipe on the first page still goes to the next page 29.7s
🟢 back gesture over a pager in a nested stack › a back swipe on the first page pops the outer stack 12.4s
🟢 back gesture over a pager in a nested stack › a forward swipe on the first page still goes to the next page 10.9s
🟢 tests/issues/issue-1083-modal-set-page.e2e.ts · 1 passed 15.4s
🟢 setPage called behind a native-stack modal lands on the requested page 15.4s
🟢 tests/issues/issue-1096-keyboard-shrink.e2e.ts · 1 passed 20.2s
🟢 keyboard avoidance does not shrink the page under the keyboard 20.2s
🟢 tests/issues/issue-1098-nested-pager.e2e.ts · 1 passed 14.7s
🟢 outer pager stays responsive after its page count changes around a nested pager 14.7s
🟢 tests/issues/issue-1142-search-bar-inset.e2e.ts · 1 passed 14.8s
🟢 FlatList inside the pager gets the search bar safe-area insets 14.8s
🟢 tests/material-top-bar.e2e.ts · 1 passed 16.8s
🟢 material top tabs survive a native-stack push and pop 16.8s
🟢 tests/nested-pager-view.e2e.ts · 1 passed 14.0s
🟢 nested horizontal and vertical pagers scroll independently 14.0s
🟢 tests/on-page-selected.e2e.ts · 1 passed 15.3s
🟢 onPageSelected fires an alert for every page change 15.3s
🟢 tests/scrollable-pager-view.e2e.ts · 1 passed 21.9s
🟢 pager inside a ScrollView keeps its page across vertical scrolls 21.9s
🟢 tests/smoke.e2e.ts · 1 passed 8.2s
🟢 horizontal pager pages with the panel buttons 8.2s
🟢 tests/tab-view-inside-scroll-view.e2e.ts · 1 passed 18.1s
🟢 TabView inside a ScrollView swipes tabs and scrolls the page 18.1s

e2e 0.21.0 · 6m 31s · ios · run artifacts

@krozniata
krozniata marked this pull request as ready for review October 26, 2026 16:00
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.

2 participants