-
Notifications
You must be signed in to change notification settings - Fork 822
ci: Upgrade golangci-linter to v2 #6739
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
ci: Upgrade golangci-linter to v2 #6739
Conversation
9892da8
to
0b49e9d
Compare
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.
Let's give it a try. Thanks!
35f5192
to
aaf5592
Compare
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.
only a tiny typo. LGTM. Thanks for waiting 🙏
pkg/util/validation/limits.go
Outdated
@@ -23,7 +23,7 @@ import ( | |||
util_log "github.com/cortexproject/cortex/pkg/util/log" | |||
) | |||
|
|||
var errMaxGlobalSeriesPerUserValidation = errors.New("The ingester.max-global-series-per-user limit is unsupported if distributor.shard-by-all-labels is disabled") | |||
var errMaxGlobalSeriesPerUserValidation = errors.New("the ingested.max-global-series-per-user limit is unsupported if distributor.shard-by-all-labels is disabled") |
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.
var errMaxGlobalSeriesPerUserValidation = errors.New("the ingested.max-global-series-per-user limit is unsupported if distributor.shard-by-all-labels is disabled") | |
var errMaxGlobalSeriesPerUserValidation = errors.New("the ingester.max-global-series-per-user limit is unsupported if distributor.shard-by-all-labels is disabled") |
pkg/ring/kv/consul/client.go
Outdated
// ErrNotFound is returned by ConsulClient.Get. | ||
ErrNotFound = fmt.Errorf("Not found") | ||
ErrNotFound = fmt.Errorf("not found") |
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.
I think this is never used, but I am not sure.
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.
I think it is safe to delete since it is never used.
aaf5592
to
3dc0291
Compare
Signed-off-by: SungJin1212 <[email protected]>
3dc0291
to
bcb1843
Compare
This PR updates
golangci-linter
version to v2 (v2.1.6). It migrates the.golangci.yml
to v2 and resolves some lint issues according to v2.Which issue(s) this PR fixes:
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]