Skip to content

Upgrade Compute to 0.1.0 and fix oagBinary build path#219

Merged
Kyle-Ye merged 5 commits intomainfrom
optimize/compute
Feb 26, 2026
Merged

Upgrade Compute to 0.1.0 and fix oagBinary build path#219
Kyle-Ye merged 5 commits intomainfrom
optimize/compute

Conversation

@Kyle-Ye
Copy link
Contributor

@Kyle-Ye Kyle-Ye commented Feb 26, 2026

Summary

  • Upgrade Compute dependency from 0.0.1/pinned revision to upstream 0.1.0 release
  • Bump OAG binary default version to 0.5.0 and update README minimum version accordingly
  • Fix oagBinary path to expose OpenAttributeGraph library product
  • Gate unit test targets behind !oagBinary (they require source access)
  • Move compatibility tests and swift-numerics dependency out of Darwin-only guard
  • Remove SSH→HTTPS git config workaround from Linux CI script

Test plan

  • Verify default source build: swift build
  • Verify Compute binary build: OPENATTRIBUTESHIMS_COMPUTE_USE_BINARY=1 swift build
  • Verify OAG binary build: OPENATTRIBUTESHIMS_OAG_USE_BINARY=1 swift build
  • Verify Linux Compute build via CI

- Bump Compute binary/source dependency from 0.0.1/rev to 0.1.0 (upstream release)
- Bump OAG binary default version to 0.5.0
- Add OpenAttributeGraph product in oagBinary path
- Gate unit test targets behind !oagBinary
- Move compatibility tests and swift-numerics out of Darwin-only gate
- Remove SSH→HTTPS git config workaround from CI script
@augmentcode
Copy link

augmentcode bot commented Feb 26, 2026

🤖 Augment PR Summary

Summary: Updates SwiftPM configuration to consume the released Compute 0.1.0 and refresh binary-build defaults.

Key changes:

• Use Compute 0.1.0 (source and binary) with updated default binary checksum.

• Bump default OpenAttributeGraph binary version to 0.5.0 and fix its env var / product exposure for oagBinary builds.

• Gate unit-test targets behind !oagBinary, and always register compatibility tests + swift-numerics dependency.

• Remove the SSH→HTTPS git URL workaround from Linux Compute CI script.

🤖 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 suggestions posted.

Fix All in Augment

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

@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 16.93%. Comparing base (7bdd424) to head (be6de46).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #219       +/-   ##
===========================================
- Coverage   32.08%   16.93%   -15.16%     
===========================================
  Files          70       72        +2     
  Lines        2596     4884     +2288     
===========================================
- Hits          833      827        -6     
- Misses       1763     4057     +2294     
Flag Coverage Δ
ios 31.48% <ø> (-0.62%) ⬇️
macos 3.48% <ø> (-6.48%) ⬇️
ubuntu 11.79% <ø> (+11.79%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 32 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

SPM requires products referencing only binary targets to be automatic
library products, not explicit dynamic.
@Kyle-Ye Kyle-Ye merged commit 0663ac4 into main Feb 26, 2026
10 checks passed
@Kyle-Ye Kyle-Ye deleted the optimize/compute branch February 26, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant