Skip to content

Optimize build-xcframework pipeline#843

Merged
Kyle-Ye merged 3 commits intomainfrom
claude/infallible-elgamal
Mar 22, 2026
Merged

Optimize build-xcframework pipeline#843
Kyle-Ye merged 3 commits intomainfrom
claude/infallible-elgamal

Conversation

@Kyle-Ye
Copy link
Collaborator

@Kyle-Ye Kyle-Ye commented Mar 22, 2026

No description provided.

…irect artifact upload

- Generate `.binaryTarget(name:path:)` entries instead of URL-based ones for
  non-tag (manual) builds since no release tag is available
- Replace upload-artifact@v4 with v7 using `archive: false` to upload a single
  xcframeworks.zip directly, avoiding the zip-of-zips issue when downloading
@Kyle-Ye Kyle-Ye changed the title Optimize build-xcframework pipeline. Optimize build-xcframework pipeline Mar 22, 2026
@github-actions github-actions bot added build issue enhancement New feature or request labels Mar 22, 2026
@augmentcode
Copy link

augmentcode bot commented Mar 22, 2026

🤖 Augment PR Summary

Summary: Adjusts the XCFramework build action to emit local-path .binaryTarget entries when no tag is provided, avoiding checksum/URL generation for manual builds.

Changes: Packages all built *.xcframework bundles into a single xcframeworks.zip and uploads it as an artifact via actions/upload-artifact@v7 (with archive: false).

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Only create individual xcframework zips when building for a release tag,
since non-tag builds don't need them.
@codecov
Copy link

codecov bot commented Mar 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.85%. Comparing base (bff879a) to head (6a35240).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #843      +/-   ##
==========================================
- Coverage   26.85%   26.85%   -0.01%     
==========================================
  Files         671      671              
  Lines       43470    43470              
==========================================
- Hits        11676    11675       -1     
- Misses      31794    31795       +1     

☔ View full report in Codecov by Sentry.
📢 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.

Use `zip -ry` instead of `zip -r` to preserve macOS framework bundle
symlinks (Versions/Current, top-level Modules/Resources). Without this,
codesigning fails because the framework structure is invalid.
@Kyle-Ye Kyle-Ye merged commit cd9a5fc into main Mar 22, 2026
7 of 8 checks passed
@Kyle-Ye Kyle-Ye deleted the claude/infallible-elgamal branch March 22, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build issue enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant