fix(agno): align opentelemetry-api minimum version with other packages#4343
fix(agno): align opentelemetry-api minimum version with other packages#4343CAN-Aidoo wants to merge 1 commit into
Conversation
All 33 sibling instrumentation packages and traceloop-sdk pin opentelemetry-api>=1.38.0,<2, but agno declared >=1.28.0,<2. This aligns the pin with the rest of the workspace so the declared minimum matches what the SDK actually requires.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated the minimum version constraint for the ChangesDependency version bump
Estimated code review effort: 1 (Trivial) | ~2 minutes Compact metadata: 1 file changed, 1 line changed (+1/-1) Suggested labels: dependencies Suggested reviewers: N/A (no reviewer information available) PoemA rabbit hops with version glee, 🚥 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. Comment |
Summary
Align the
opentelemetry-apidependency pin inopentelemetry-instrumentation-agnowith the rest of the workspace:>=1.28.0,<2→>=1.38.0,<2.Motivation
All 33 sibling instrumentation packages and
traceloop-sdkitself pinopentelemetry-api>=1.38.0,<2. The lower pin inagnowas misleading — it advertised support for OTel API versions the package isn't tested against, and since any real install pulls the SDK's>=1.38.0anyway, the declaration was purely informational drift.Test plan
grep 'opentelemetry-api>=' packages/*/pyproject.toml— agno now matches the other 33 packages.pyproject.tomlmetadata changes; no Python code paths affected.Summary by CodeRabbit