Closed henrybear327 closed 3 days ago
Passing the dependency update torch to @ArkaSaha30 for the entire July after this one! :)
Updating golangci-lint
to 1.59.1 generated the following lint issue
FAIL: (code:1):
% (cd server && golangci-lint run --config /Users/henrybear327/go/src/etcd/tools/.golangci.yaml ./...)
storage/mvcc/kvstore.go:312:27: (*store).restore - result 0 (error) is always nil (unparam)
func (s *store) restore() error {
^
FAIL: 'run golangci-lint run --config /Users/henrybear327/go/src/etcd/tools/.golangci.yaml ./...' checking failed (!=0 return code)
There was a Failure in module server, aborting...
FAIL: 'lint' FAILED at Mon Jun 24 21:48:39 CEST 2024
make: *** [verify-lint] Error 255
Both Attach()
and compactLockfree()
within the function restore()
are able to return an error
but we only log them in the current implementation, thus, the return value for error
is always nil
, hence the linter warning.
@ahrtr do we need to do anything about this? Or we should just suppress the linter warning here :) Thanks
@henrybear327 suggest to only suppress the error for now.
Thanks
Both
Attach()
andcompactLockfree()
within the functionrestore()
are able to return anerror
but we only log them in the current implementation, thus, the return value forerror
is alwaysnil
, hence the linter warning.
Done as agreed.
@jmhbnz @ahrtr please close the indirect dependency PRs :) Thanks
@jmhbnz @ahrtr please close the indirect dependency PRs :) Thanks
I have been thinking all members have permissions to do that (close dependabot PRs). Can you double check it?
Sadly, no, otherwise I would have done it directly in the past few weeks :D
Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.
This pull request completes this week's etcd dependency updates following our dependency roster and dependency management instructions.
Summary of actions