feat: update grpc and go#252
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughUpdated Go module toolchain from version 1.23.0 to 1.25.0 with toolchain set to go1.26.1. Upgraded multiple dependencies including protoc-gen-validate, gRPC, protobuf, and related Google libraries to newer versions, plus indirect transitive dependencies. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 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 |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Updates the proto Go module’s toolchain/Go version settings and refreshes gRPC/protobuf-related dependencies accordingly.
Changes:
- Bump
protomodulegoversion and add atoolchaindirective. - Upgrade gRPC/protobuf/genproto and protoc-gen-validate versions.
- Regenerate
proto/go.sumto reflect the updated dependency graph.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| proto/go.mod | Updates minimum Go version/toolchain and bumps core gRPC/protobuf-related dependencies. |
| proto/go.sum | Refreshes dependency checksums after module version updates. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@proto/go.mod`:
- Line 8: The buf generator version for validate-go is out of sync with the
runtime: proto/go.mod uses protoc-gen-validate v1.3.0 but buf.gen.yaml pins
validate-go:v1.0.4; update the generator to match by changing the validate-go
image tag in buf.gen.yaml (the entry at/around buf.gen.yaml:21) from v1.0.4 to
v1.3.0 so the generated code and runtime versions align with the dependency in
proto/go.mod.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 17a4cc8c-9715-4730-8b58-a1fe89738662
⛔ Files ignored due to path filters (1)
proto/go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
proto/go.mod
…alidate v1.3.0 output
Bump grpc v1.81.1, protobuf v1.36.11, grpc-gateway v2.29.0, protoc-gen-validate v1.3.3, genproto api/rpc to latest, and toolchain go1.26.4. Align buf.gen.yaml plugin versions and regenerate all generated code.
Description
What problem is being solved?
How is it being solved?
What changes are made to solve it?
References
Review Checklist
mainSummary by CodeRabbit