adapters/wredis: bump testcontainers-go to v0.43.0#198
Conversation
Old testcontainers-go v0.40.0 pulled in outdated github.com/docker/docker, github.com/containerd/containerd, and go.opentelemetry.io/otel — the source of ~20 open Dependabot alerts (several critical) on this repo. v0.43.0 replaces the docker/containerd deps with the lighter moby/moby/api + moby/moby/client split and bumps otel to v1.44.0, clearing them. Test-only dependency (testcontainers spins up Redis for integration tests); no change to the adapter's runtime behavior. All adapters/wredis tests pass. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated the Redis adapter’s Testcontainers and Redis module dependencies from v0.40.0 to v0.43.0. The change also refreshes numerous indirect Docker, OpenTelemetry, Prometheus, and Go ecosystem dependencies, while removing several no-longer-listed indirect modules. Estimated code review effort: 1 (Trivial) | ~5 minutes Poem
🚥 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
testcontainers-go(and themodules/redissubmodule) from v0.40.0 to v0.43.0 inadapters/wredis.docker/dockerandcontainerd/containerdare replaced by the newer, lightermoby/moby/api+moby/moby/clientsplit, andgo.opentelemetry.io/otelmoves to v1.44.0.testcontainers-gois a test-only dependency here (used to spin up a Redis container for integration tests) — no change to the adapter's shipped runtime behavior.Test plan
go build ./...go vet ./...go test -v -timeout=10m ./...— all tests pass locally against Docker (Rancher Desktop)Companion PR for
adapters/kafkastreamer's equivalent bump is separate, since that one currently has a testcontainers Kafka-module issue under investigation.🤖 Generated with Claude Code
Summary by CodeRabbit