Repository navigation
chore(deps): bump Go to 1.26 everywhere - #3940
Conversation
✅ Deploy Preview for testcontainers-go ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (7)
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review. Summary by CodeRabbit
WalkthroughThe repository now targets Go 1.26 across its modules, build environments, and CI configuration. The Go-version update script resolves patch versions and image digests and updates additional version references. Several code changes adjust reflection checks, logging, HTTP dialing, test data, and expression syntax. ChangesGo 1.26 rollout
Go code adjustments
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The Go version updates align across the reviewed modules and CI, with no identified merge-blocking behavior. The PR is ready to merge after normal checks. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The reviewed changes are concentrated in build and test environments. Existing CI permission boundaries and Go builder-image pinning are preserved. Version-update automation can leave partially synchronized configuration after failure or interruption, but no introduced authority escalation was established. Broader runtime coverage remains incomplete. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
Hardening Proposals
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 54.84% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 31 functions across 25 files. (7 skipped: 7 unsupported.)
✨ 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. A rabbit checks the Go-version trail, Comment |
e6c15a9 to
f75fdf3
Compare
Go 1.25 reached EOL with the 1.27 release. - go.mod: go 1.26.0, toolchain go1.26.8; CI matrix 1.26.x and 1.27.x - golangci-lint v2.14.0 (built with Go 1.26) and fixes for its new findings - drop the Go 1.25 skip for weaviate in ci-test-go.yml - bump-go.sh: fix major.minor matching, handle toolchain, Dockerfile digests, circle_ci.md patch versions and AI.md Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
f75fdf3 to
0de3222
Compare
What does this PR do?
go.modfiles (go 1.26.0,toolchain go1.26.8).golang:1.26-alpineby digest), devcontainer and AI.md.scripts/bump-go.sh: it compared the three-part go.mod version against the two-part version used everywhere else, so only go.mod lines were bumped. It now also handlestoolchain, Dockerfile digests, the CircleCI patch versions and AI.md.Why is it important?
Go 1.25 reached end of life with the Go 1.27 release (https://endoflife.date/go).
Related issues
How to test this PR
make tidy-allproduces no changes.make lintin the core module andgolangci-lint runin every module pass with v2.14.0../scripts/bump-go.sh 1.27.🤖 Generated with Claude Code