-
Notifications
You must be signed in to change notification settings - Fork 11
fix(sdk): endpoint building #23
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.
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,6 @@ | ||
| go 1.23 | ||
| go 1.24.0 | ||
|
|
||
| toolchain go1.24.6 | ||
|
|
||
| use ( | ||
| sample-app | ||
|
|
||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,51 +1,56 @@ | ||
| module github.com/traceloop/go-openllmetry/traceloop-sdk | ||
|
|
||
| go 1.23 | ||
| go 1.24.0 | ||
|
|
||
| toolchain go1.24.6 | ||
|
|
||
| require ( | ||
| github.com/kluctl/go-jinja2 v0.0.0-20240108142937-8839259d2537 | ||
| github.com/sashabaranov/go-openai v1.18.1 | ||
| github.com/traceloop/go-openllmetry/semconv-ai v0.0.0-20250405130248-6b2b4b41102b | ||
| github.com/kluctl/go-jinja2 v0.0.0-20241217133422-164d7f6ac307 | ||
| github.com/sashabaranov/go-openai v1.41.1 | ||
| github.com/traceloop/go-openllmetry/semconv-ai v0.0.0-20250827154028-23d2bf930621 | ||
| go.opentelemetry.io/otel v1.37.0 | ||
| go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.22.0 | ||
| go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.37.0 | ||
| go.opentelemetry.io/otel/trace v1.37.0 | ||
| ) | ||
|
|
||
| require ( | ||
| github.com/cenkalti/backoff/v4 v4.2.1 // indirect | ||
| github.com/Masterminds/semver/v3 v3.4.0 // indirect | ||
| github.com/cenkalti/backoff/v4 v4.3.0 // indirect | ||
| github.com/cenkalti/backoff/v5 v5.0.3 // indirect | ||
| github.com/go-logr/logr v1.4.3 // indirect | ||
|
Comment on lines
+17
to
20
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainMultiple major versions of cenkalti/backoff pulled in. You have v2 (direct), v4 (indirect), and v5 (indirect). This increases bloat and can confuse readers. If you don’t import Use this to confirm usage and prune: Also applies to: 37-37 🏁 Script executed: #!/bin/bash
# Find imports of all backoff module paths
rg -nP -C1 '\bgithub\.com/cenkalti/backoff(/v[45])?\b'
go mod tidyLength of output: 2264 Migrate backoff usage to v5 and drop v2 require
🤖 Prompt for AI Agents |
||
| github.com/go-logr/stdr v1.2.2 // indirect | ||
| github.com/golang/protobuf v1.5.3 // indirect | ||
| github.com/gofrs/flock v0.12.1 // indirect | ||
| github.com/golang/protobuf v1.5.4 // indirect | ||
| github.com/google/uuid v1.6.0 // indirect | ||
| github.com/grpc-ecosystem/grpc-gateway/v2 v2.16.0 // indirect | ||
| github.com/grpc-ecosystem/grpc-gateway/v2 v2.27.2 // indirect | ||
| go.opentelemetry.io/auto/sdk v1.1.0 // indirect | ||
| go.opentelemetry.io/otel/metric v1.37.0 // indirect | ||
| go.opentelemetry.io/proto/otlp v1.0.0 // indirect | ||
| golang.org/x/text v0.14.0 // indirect | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20231002182017-d307bd883b97 // indirect | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20231002182017-d307bd883b97 // indirect | ||
| google.golang.org/grpc v1.60.1 // indirect | ||
| google.golang.org/protobuf v1.32.0 // indirect | ||
| go.opentelemetry.io/proto/otlp v1.7.1 // indirect | ||
| golang.org/x/text v0.28.0 // indirect | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20250826171959-ef028d996bc1 // indirect | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20250826171959-ef028d996bc1 // indirect | ||
| google.golang.org/grpc v1.75.0 // indirect | ||
| google.golang.org/protobuf v1.36.8 // indirect | ||
| ) | ||
|
|
||
| require ( | ||
| github.com/cenkalti/backoff v2.2.1+incompatible | ||
| github.com/go-git/gcfg v1.5.1-0.20230307220236-3a3c6141e376 // indirect | ||
| github.com/go-git/go-billy/v5 v5.5.0 // indirect | ||
| github.com/go-git/go-git/v5 v5.11.0 // indirect | ||
| github.com/go-git/go-billy/v5 v5.6.2 // indirect | ||
| github.com/go-git/go-git/v5 v5.16.2 // indirect | ||
| github.com/gobwas/glob v0.2.3 // indirect | ||
| github.com/hashicorp/errwrap v1.1.0 // indirect | ||
| github.com/hashicorp/go-multierror v1.1.1 // indirect | ||
| github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 // indirect | ||
| github.com/jinzhu/copier v0.4.0 // indirect | ||
| github.com/kluctl/go-embed-python v0.0.0-3.11.6-20231002-1 // indirect | ||
| github.com/rogpeppe/go-internal v1.13.1 // indirect | ||
| github.com/kluctl/go-embed-python v0.0.0-3.13.1-20241219-1 // indirect | ||
| github.com/rogpeppe/go-internal v1.14.1 // indirect | ||
| github.com/sirupsen/logrus v1.9.3 // indirect | ||
| go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.22.0 | ||
| go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.22.0 | ||
| go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.37.0 | ||
| go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.37.0 | ||
| go.opentelemetry.io/otel/sdk v1.37.0 | ||
| golang.org/x/net v0.19.0 // indirect | ||
| golang.org/x/sync v0.5.0 // indirect | ||
| golang.org/x/sys v0.33.0 // indirect | ||
| golang.org/x/net v0.43.0 // indirect | ||
| golang.org/x/sync v0.16.0 // indirect | ||
| golang.org/x/sys v0.35.0 // indirect | ||
| gopkg.in/warnings.v0 v0.1.2 // indirect | ||
| ) | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Go 1.24 toolchain: verify CI/build images.
Confirm your CI/docker/tooling uses Go ≥ 1.24.6; otherwise builds will fail.
Run:
🏁 Script executed:
Length of output: 145
Update Go toolchain to ≥1.24.6 in CI and local environments
go versionreports go1.24.1; bump all CI/docker images and any local Go installs to 1.24.6 to match thego.modtoolchain and avoid build failures.🤖 Prompt for AI Agents