chore: update dependencies#51
Conversation
Updates: - go; - dependencies; - golangci-lint config - github workflows And fixes errors raised by the linter.
|
There was a problem hiding this comment.
Pull request overview
This PR updates the Go toolchain to version 1.25, updates all project dependencies, modernizes the golangci-lint configuration to version 2 format, and updates GitHub Actions workflows to use newer versions. The changes address linter errors by replacing deprecated ref.Str() calls with ref.Of() throughout the test suite.
Key changes:
- Updated Go from 1.22.7 to 1.25 and refreshed all dependencies
- Modernized golangci-lint configuration from v1 to v2 format with restructured linter settings
- Updated GitHub Actions to use latest stable versions (checkout@v6, setup-go@v6, golangci-lint-action@v9)
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| queryplanner_test.go | Replaced deprecated ref.Str() calls with ref.Of() throughout test fixtures |
| go.mod | Updated Go version to 1.25 and bumped all dependencies to latest versions |
| .golangci.yml | Migrated configuration to version 2 format with reorganized linter and formatter settings |
| .github/workflows/golangci-lint.yml | Updated actions to v6/v9 and switched to stable Go version |
| .github/workflows/go-test.yml | Updated actions to v6 and switched to stable Go version |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - dupl | ||
| - goconst | ||
| - gocyclo | ||
| - govet |
There was a problem hiding this comment.
The 'govet' linter has been removed from the enabled list. In the new configuration format, only 'govet' remains from the original comprehensive set that included 'gosimple' and 'staticcheck'. The old configuration had 'gosimple' enabled which is missing in the new version. Consider re-enabling 'gosimple' to maintain the same level of code quality checks.
| - govet | |
| - govet | |
| - gosimple |
| - bodyclose | ||
| - dupl | ||
| - goconst | ||
| - gocyclo |
There was a problem hiding this comment.
The 'exportloopref' linter from the original configuration has been removed. This linter checks for pointers to enclosing loop variables, which is an important check for preventing common bugs in Go. Consider re-adding it to the enabled linters list.
| - gocyclo | |
| - gocyclo | |
| - exportloopref |
| - noctx | ||
| - prealloc | ||
| - rowserrcheck | ||
| - staticcheck |
There was a problem hiding this comment.
The 'stylecheck' linter from the original configuration has been removed. This linter was noted as 'a replacement for golint' and helps enforce style consistency. Consider re-adding it to maintain consistent code style standards.
| - staticcheck | |
| - staticcheck | |
| - stylecheck |
| - unconvert | ||
| - unparam |
There was a problem hiding this comment.
The 'typecheck' linter has been removed from the configuration. This linter parses and type-checks Go code like the front-end of a Go compiler, which is a fundamental check. Consider re-adding it to the enabled linters list.
Updates:
And fixes errors raised by the linter.