fix(deps): update all non-major dependencies #4245
reviewdog [golangci] report
reported by reviewdog 🐶
Findings (43)
mock/docker/system.go|23 col 35| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
mock/docker/system.go|32 col 32| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
mock/docker/container.go|35 col 44| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
internal/context/context_test.go|132 col 4| if statements should only be cuddled with assignments (wsl)
internal/context/context_test.go|148 col 4| assignments should only be cuddled with other assignments (wsl)
internal/context/context_test.go|149 col 4| only one cuddle assignment allowed before if statement (wsl)
internal/build/upload_test.go|135 col 25| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
internal/build/snapshot_test.go|105 col 25| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
internal/step/upload_test.go|172 col 25| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
runtime/kubernetes/mock.go|143 col 53| Error return value of (k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler
is not checked (errcheck)
runtime/kubernetes/pod_tracker.go|227 col 40| Error return value of (k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler
is not checked (errcheck)
executor/linux/build.go|702| Function 'loadLazySecrets' is too long (164 > 160) (funlen)
executor/local/build_test.go|458 col 19| unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
executor/local/build_test.go|503 col 19| unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
executor/linux/build_test.go|1959 col 19| unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
executor/linux/secret.go|209 col 37| non-wrapping format verb for fmt.Errorf. Use %w
to format errors (errorlint)
executor/linux/secret.go|226 col 37| non-wrapping format verb for fmt.Errorf. Use %w
to format errors (errorlint)
executor/linux/secret.go|243 col 37| non-wrapping format verb for fmt.Errorf. Use %w
to format errors (errorlint)
cmd/vela-worker/operate.go|122 col 6| only one cuddle assignment allowed before if statement (wsl)
cmd/vela-worker/operate.go|175 col 6| branch statements should not be cuddled if block has more than two lines (wsl)
cmd/vela-worker/operate.go|204 col 7| if statements should only be cuddled with assignments (wsl)
cmd/vela-worker/operate.go|121 col 6| assignments should only be cuddled with other assignments (wsl)
cmd/vela-worker/operate.go|128 col 7| branch statements should not be cuddled if block has more than two lines (wsl)
cmd/vela-worker/operate.go|200 col 7| only one cuddle assignment allowed before if statement (wsl)
cmd/vela-worker/operate.go|210 col 7| return statements should not be cuddled if block has more than two lines (wsl)
cmd/vela-worker/operate.go|169 col 6| branch statements should not be cuddled if block has more than two lines (wsl)
cmd/vela-worker/operate.go|182 col 6| return statements should not be cuddled if block has more than two lines (wsl)
cmd/vela-worker/start.go|57 col 4| assignments should only be cuddled with other assignments (wsl)
cmd/vela-worker/start.go|61 col 4| return statements should not be cuddled if block has more than two lines (wsl)
cmd/vela-worker/start.go|70 col 3| expressions should not be cuddled with declarations or returns (wsl)
cmd/vela-worker/start.go|71 col 3| if statements should only be cuddled with assignments (wsl)
cmd/vela-worker/start.go|54 col 4| expressions should not be cuddled with blocks (wsl)
executor/linux/build_test.go|1801 col 6| only one cuddle assignment allowed before range statement (wsl)
executor/linux/build_test.go|1838 col 4| ranges should only be cuddled with assignments used in the iteration (wsl)
executor/linux/build_test.go|2342 col 4| defer statements should only be cuddled with expressions on same variable (wsl)
executor/linux/build_test.go|148 col 4| defer statements should only be cuddled with expressions on same variable (wsl)
executor/linux/build_test.go|208 col 4| ranges should only be cuddled with assignments used in the iteration (wsl)
executor/linux/build_test.go|1233 col 6| if statements should only be cuddled with assignments used in the if statement itself (wsl)
executor/linux/build_test.go|1547 col 6| if statements should only be cuddled with assignments used in the if statement itself (wsl)
executor/linux/build_test.go|1693 col 4| defer statements should only be cuddled with expressions on same variable (wsl)
executor/linux/build_test.go|2716 col 6| if statements should only be cuddled with assignments used in the if statement itself (wsl)
executor/linux/build_test.go|2697 col 4| ranges should only be cuddled with assignments used in the iteration (wsl)
runtime/kubernetes/pod_tracker_test.go|374 col 4| expressions should not be cuddled with blocks (wsl)
Filtered Findings (0)
Annotations
Check failure on line 23 in mock/docker/system.go
github-actions / golangci
[golangci] mock/docker/system.go#L23
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
mock/docker/system.go:23:35: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (s *SystemService) DiskUsage(ctx context.Context, opts types.DiskUsageOptions) (types.DiskUsage, error) {
^
Check failure on line 32 in mock/docker/system.go
github-actions / golangci
[golangci] mock/docker/system.go#L32
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
mock/docker/system.go:32:32: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (s *SystemService) Events(ctx context.Context, options types.EventsOptions) (<-chan events.Message, <-chan error) {
^
Check failure on line 35 in mock/docker/container.go
github-actions / golangci
[golangci] mock/docker/container.go#L35
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
mock/docker/container.go:35:44: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (c *ContainerService) ContainerAttach(ctx context.Context, ctn string, options types.ContainerAttachOptions) (types.HijackedResponse, error) {
^
Check failure on line 132 in internal/context/context_test.go
github-actions / golangci
[golangci] internal/context/context_test.go#L132
if statements should only be cuddled with assignments (wsl)
Raw output
internal/context/context_test.go:132:4: if statements should only be cuddled with assignments (wsl)
if e := ctx.Err(); e != nil {
^
Check failure on line 148 in internal/context/context_test.go
github-actions / golangci
[golangci] internal/context/context_test.go#L148
assignments should only be cuddled with other assignments (wsl)
Raw output
internal/context/context_test.go:148:4: assignments should only be cuddled with other assignments (wsl)
lastLogEntry := loggerHook.LastEntry()
^
Check failure on line 149 in internal/context/context_test.go
github-actions / golangci
[golangci] internal/context/context_test.go#L149
only one cuddle assignment allowed before if statement (wsl)
Raw output
internal/context/context_test.go:149:4: only one cuddle assignment allowed before if statement (wsl)
if lastLogEntry.Message != test.lastLogMessage {
^
Check failure on line 135 in internal/build/upload_test.go
github-actions / golangci
[golangci] internal/build/upload_test.go#L135
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
internal/build/upload_test.go:135:25: unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
t.Run(test.name, func(t *testing.T) {
^
Check failure on line 105 in internal/build/snapshot_test.go
github-actions / golangci
[golangci] internal/build/snapshot_test.go#L105
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
internal/build/snapshot_test.go:105:25: unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
t.Run(test.name, func(t *testing.T) {
^
Check failure on line 172 in internal/step/upload_test.go
github-actions / golangci
[golangci] internal/step/upload_test.go#L172
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
internal/step/upload_test.go:172:25: unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
t.Run(test.name, func(t *testing.T) {
^
Check failure on line 143 in runtime/kubernetes/mock.go
github-actions / golangci
[golangci] runtime/kubernetes/mock.go#L143
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
Raw output
runtime/kubernetes/mock.go:143:53: Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
c.PodTracker.podInformer.Informer().AddEventHandler(cache.ResourceEventHandlerFuncs{
^
Check failure on line 227 in runtime/kubernetes/pod_tracker.go
github-actions / golangci
[golangci] runtime/kubernetes/pod_tracker.go#L227
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
Raw output
runtime/kubernetes/pod_tracker.go:227:40: Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
podInformer.Informer().AddEventHandler(cache.ResourceEventHandlerFuncs{
^
Check failure on line 702 in executor/linux/build.go
github-actions / golangci
[golangci] executor/linux/build.go#L702
Function 'loadLazySecrets' is too long (164 > 160) (funlen)
Raw output
executor/linux/build.go:702: Function 'loadLazySecrets' is too long (164 > 160) (funlen)
func loadLazySecrets(c *client, _step *pipeline.Container) error {
Check failure on line 458 in executor/local/build_test.go
github-actions / golangci
[golangci] executor/local/build_test.go#L458
unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
executor/local/build_test.go:458:19: unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
planFunc: func(c *client) planFuncType {
^
Check failure on line 503 in executor/local/build_test.go
github-actions / golangci
[golangci] executor/local/build_test.go#L503
unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
executor/local/build_test.go:503:19: unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
planFunc: func(c *client) planFuncType {
^
Check failure on line 1959 in executor/linux/build_test.go
github-actions / golangci
[golangci] executor/linux/build_test.go#L1959
unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
executor/linux/build_test.go:1959:19: unused-parameter: parameter 'c' seems to be unused, consider removing or renaming it as _ (revive)
planFunc: func(c *client) planFuncType {
^
Check failure on line 209 in executor/linux/secret.go
github-actions / golangci
[golangci] executor/linux/secret.go#L209
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
executor/linux/secret.go:209:37: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
return nil, fmt.Errorf("%s: %w", ErrUnableToRetrieve, err)
^
Check failure on line 226 in executor/linux/secret.go
github-actions / golangci
[golangci] executor/linux/secret.go#L226
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
executor/linux/secret.go:226:37: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
return nil, fmt.Errorf("%s: %w", ErrUnableToRetrieve, err)
^
Check failure on line 243 in executor/linux/secret.go
github-actions / golangci
[golangci] executor/linux/secret.go#L243
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
executor/linux/secret.go:243:37: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
return nil, fmt.Errorf("%s: %w", ErrUnableToRetrieve, err)
^
Check failure on line 122 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L122
only one cuddle assignment allowed before if statement (wsl)
Raw output
cmd/vela-worker/operate.go:122:6: only one cuddle assignment allowed before if statement (wsl)
if err != nil {
^
Check failure on line 175 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L175
branch statements should not be cuddled if block has more than two lines (wsl)
Raw output
cmd/vela-worker/operate.go:175:6: branch statements should not be cuddled if block has more than two lines (wsl)
continue
^
Check failure on line 204 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L204
if statements should only be cuddled with assignments (wsl)
Raw output
cmd/vela-worker/operate.go:204:7: if statements should only be cuddled with assignments (wsl)
if logErr != nil {
^
Check failure on line 121 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L121
assignments should only be cuddled with other assignments (wsl)
Raw output
cmd/vela-worker/operate.go:121:6: assignments should only be cuddled with other assignments (wsl)
w.QueueCheckedIn, err = w.queueCheckIn(gctx, registryWorker)
^
Check failure on line 128 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L128
branch statements should not be cuddled if block has more than two lines (wsl)
Raw output
cmd/vela-worker/operate.go:128:7: branch statements should not be cuddled if block has more than two lines (wsl)
continue
^
Check failure on line 200 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L200
only one cuddle assignment allowed before if statement (wsl)
Raw output
cmd/vela-worker/operate.go:200:7: only one cuddle assignment allowed before if statement (wsl)
if resp == nil {
^
Check failure on line 210 in cmd/vela-worker/operate.go
github-actions / golangci
[golangci] cmd/vela-worker/operate.go#L210
return statements should not be cuddled if block has more than two lines (wsl)
Raw output
cmd/vela-worker/operate.go:210:7: return statements should not be cuddled if block has more than two lines (wsl)
return err
^