-
Notifications
You must be signed in to change notification settings - Fork 29
chore: go bump 1.25 #666
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
chore: go bump 1.25 #666
Conversation
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.
Pull Request Overview
This PR upgrades the Go version from 1.24.3 to 1.25.0 across the project and updates dependencies to maintain compatibility with the new Go version. The upgrade also includes updates to golangci-lint configuration and GitHub workflow configurations.
- Upgraded Go version from 1.24.3 to 1.25.0 in go.mod and Dockerfile
- Updated golangci-lint configuration to version 2 format with restructured linter settings
- Added nolint directives to suppress revive and noctx warnings introduced by the new linter version
Reviewed Changes
Copilot reviewed 21 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Updated Go version to 1.25.0 and bumped various dependencies |
| Dockerfile | Updated base image to golang:1.25-alpine |
| .golangci.yml | Migrated to golangci-lint v2 configuration format |
| .github/workflows/*.yml | Updated Go version and golangci-lint version in CI workflows |
| Multiple Go files | Added nolint:revive directives to suppress package comment warnings |
| testutil/*.go, service/server.go | Added nolint:noctx directives for context-related warnings |
| CHANGELOG.md | Added entry documenting the Go version bump |
Comments suppressed due to low confidence (1)
.golangci.yml:14
- golangci-lint version v2.4.0 does not exist. The latest version as of my knowledge cutoff is v1.x.x series. Please verify this version exists or use a valid golangci-lint version.
- containedctx
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
fix lint tho |
(cherry picked from commit 79281ac)
<hr>This is an automatic backport of pull request #666 done by [Mergify](https://mergify.com). Co-authored-by: Lazar <12626340+Lazar955@users.noreply.github.com>
No description provided.