chore: bump go-coldbrew dependencies to latest versions#22
Conversation
core v0.1.33, errors v0.2.5, log v0.2.7, interceptors v0.1.12, options v0.2.6, tracing v0.1.0, hystrixprometheus v0.1.2. Includes Phase 4 migration (providers/prometheus, SupportPackageIsVersion1 sentinels).
|
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)
📝 WalkthroughWalkthroughThis PR updates the Go module dependencies by removing several direct requirements and pinning newer versions of core packages including go-coldbrew modules and google.golang.org libraries, while restructuring the indirect dependency graph across tooling, protobuf, and tracing-related packages. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Pull request overview
Updates the cookiecutter Go template to use newer versions of the go-coldbrew framework modules, keeping generated services aligned with the latest upstream releases.
Changes:
- Bump
github.com/go-coldbrew/{core,errors,log}to newer versions. - Bump indirect
go-coldbrewdependencies (interceptors,options,tracing, andhystrixprometheus) to newer versions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…w v0.1.33 Regenerated the complete go.mod from a clean go mod tidy run. Bumps buf to v1.66.1, coldbrew core to v0.1.33, and all transitive dependencies to their resolved versions. Removes stale deps (go-grpc-prometheus, NYTimes/gziphandler, etc.). Tested locally: go mod download + go tool buf generate works without 'updates to go.mod needed' error.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| github.com/go-coldbrew/log v0.2.7 | ||
| github.com/kelseyhightower/envconfig v1.4.0 | ||
| github.com/planetscale/vtprotobuf v0.6.1-0.20240319094008-0393e58bdf10 | ||
| github.com/stretchr/testify v1.11.1 |
There was a problem hiding this comment.
main.go imports github.com/grpc-ecosystem/grpc-gateway/v2/runtime, but github.com/grpc-ecosystem/grpc-gateway/v2 is no longer required anywhere in go.mod. This will force go to rewrite go.mod/go.sum during builds (or fail under -mod=readonly) and makes the template’s dependency list incomplete. Add grpc-gateway/v2 back as a direct requirement (or run go mod tidy after ensuring the template still builds) so the module graph matches the code imports.
| github.com/stretchr/testify v1.11.1 | |
| github.com/stretchr/testify v1.11.1 | |
| github.com/grpc-ecosystem/grpc-gateway/v2 v2.20.0 |
Summary
Test plan
Summary by CodeRabbit