Skip to content

ci: cache xcframework builds #5449

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 13 commits into from
Jun 23, 2025
Merged

ci: cache xcframework builds #5449

merged 13 commits into from
Jun 23, 2025

Conversation

@armcknight
Copy link
Member

@armcknight armcknight commented Jun 20, 2025

After parallelizing the build for #4925, one of the proposed further improvements was to cache build products. This tries to do that.

#skip-changelog

@armcknight armcknight changed the title Armcknight/ci/cache xcframework builds ci: cache xcframework builds Jun 20, 2025
@armcknight armcknight mentioned this pull request Jun 20, 2025
Merged
itaybre
itaybre reviewed Jun 23, 2025
View reviewed changes
@armcknight armcknight mentioned this pull request Jun 23, 2025
Closed
3 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Jun 23, 2025

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1219.37 ms 1250.92 ms 31.55 ms
Size 23.75 KiB 850.72 KiB 826.97 KiB

@armcknight armcknight merged commit 5cfc768 into main Jun 23, 2025
69 of 73 checks passed
@armcknight armcknight deleted the armcknight/ci/cache-xcframework-builds branch June 23, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@itaybre itaybre itaybre approved these changes

@philipphofmann philipphofmann Awaiting requested review from philipphofmann philipphofmann is a code owner

@philprime philprime Awaiting requested review from philprime philprime is a code owner

@noahsmartin noahsmartin Awaiting requested review from noahsmartin noahsmartin is a code owner

@trevor-e trevor-e Awaiting requested review from trevor-e

@NicoHinderling NicoHinderling Awaiting requested review from NicoHinderling

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@armcknight @itaybre