Repository navigation
fix(deps): bump SDK to v0.33.0 in /service and /examples - #4109
Conversation
Signed-off-by: Elizabeth Healy <35498075+elizabethhealy@users.noreply.github.com> Co-authored-by: CoopAgent <coopagent@users.noreply.github.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe examples and service modules now require ChangesSDK Dependency Updates
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The documented reader-position change does not affect the inspected example inputs, and no concrete service regression was established. Merge risk is minimal; normal build validation remains outstanding. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit checks the versions in a row Comment |
Benchmark results, click to expandBenchmark authorization.GetDecisions Results:
Benchmark authorization.v2.GetMultiResourceDecision Results:
Benchmark Statistics
Bulk Benchmark Results
TDF3 Benchmark Results:
|
|
Bump the SDK dependency to v0.33.0 (released in #4068).
service:github.com/opentdf/platform/sdkv0.31.0 → v0.33.0examples:github.com/opentdf/platform/sdkv0.32.0 → v0.33.0go.sumchecksums updated in both (verified against sum.golang.org)SDK v0.33.0 transitive deps are already at or below the versions both modules pin, so no other module changes are needed.
Not included:
test/integrationuses a localreplace, so nothing to bumptests-bddis still pinned to sdk v0.25.0 / service v0.7.2 with many older deps. Upgrading it needs a fullgo mod tidy, so it should get its own PR.Summary by CodeRabbit