Skip to content

chore(deps): update all non-major dependencies #4220

chore(deps): update all non-major dependencies

chore(deps): update all non-major dependencies #4220

GitHub Actions / golangci failed Mar 5, 2024 in 1s

reviewdog [golangci] report

reported by reviewdog 🐶

Findings (42)

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)
runtime/docker/build.go|13 col 31| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
runtime/docker/build.go|21 col 29| unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
runtime/docker/build.go|29 col 30| unused-parameter: parameter 'ctx' 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)
runtime/kubernetes/pod_tracker_test.go|185 col 25| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
runtime/kubernetes/pod_tracker_test.go|259 col 25| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
runtime/kubernetes/pod_tracker_test.go|328 col 25| unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
runtime/kubernetes/pod_tracker_test.go|374 col 4| expressions should not be cuddled with blocks (wsl)
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|202 col 37| non-wrapping format verb for fmt.Errorf. Use %w to format errors (errorlint)
executor/linux/secret.go|219 col 37| non-wrapping format verb for fmt.Errorf. Use %w to format errors (errorlint)
executor/linux/secret.go|236 col 37| non-wrapping format verb for fmt.Errorf. Use %w to format errors (errorlint)
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|175 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/operate.go|200 col 7| only one cuddle assignment allowed before if statement (wsl)
cmd/vela-worker/operate.go|204 col 7| if statements should only be cuddled with assignments (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|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|122 col 6| only one cuddle assignment allowed before if statement (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|50 col 4| assignments should only be cuddled with other assignments (wsl)
cmd/vela-worker/start.go|54 col 4| expressions should not be cuddled with blocks (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|2463 col 6| if statements should only be cuddled with assignments used in the if statement itself (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|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|1801 col 6| only one cuddle assignment allowed before range statement (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|1838 col 4| ranges should only be cuddled with assignments used in the iteration (wsl)
executor/linux/build_test.go|2697 col 4| ranges should only be cuddled with assignments used in the iteration (wsl)
executor/linux/build_test.go|208 col 4| ranges should only be cuddled with assignments used in the iteration (wsl)

Filtered Findings (0)

Annotations

Check failure on line 132 in internal/context/context_test.go

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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 13 in runtime/docker/build.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/docker/build.go#L13

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
runtime/docker/build.go:13:31: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (c *client) InspectBuild(ctx context.Context, b *pipeline.Build) ([]byte, error) {
                              ^

Check failure on line 21 in runtime/docker/build.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/docker/build.go#L21

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
runtime/docker/build.go:21:29: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (c *client) SetupBuild(ctx context.Context, b *pipeline.Build) error {
                            ^

Check failure on line 29 in runtime/docker/build.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/docker/build.go#L29

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
runtime/docker/build.go:29:30: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (c *client) StreamBuild(ctx context.Context, b *pipeline.Build) error {
                             ^

Check failure on line 143 in runtime/kubernetes/mock.go

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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 185 in runtime/kubernetes/pod_tracker_test.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/kubernetes/pod_tracker_test.go#L185

unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
runtime/kubernetes/pod_tracker_test.go:185: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 259 in runtime/kubernetes/pod_tracker_test.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/kubernetes/pod_tracker_test.go#L259

unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
runtime/kubernetes/pod_tracker_test.go:259: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 328 in runtime/kubernetes/pod_tracker_test.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/kubernetes/pod_tracker_test.go#L328

unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
Raw output
runtime/kubernetes/pod_tracker_test.go:328: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 374 in runtime/kubernetes/pod_tracker_test.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] runtime/kubernetes/pod_tracker_test.go#L374

expressions should not be cuddled with blocks (wsl)
Raw output
runtime/kubernetes/pod_tracker_test.go:374:4: expressions should not be cuddled with blocks (wsl)
			tracker.Stop()
			^

Check failure on line 458 in executor/local/build_test.go

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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 202 in executor/linux/secret.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] executor/linux/secret.go#L202

non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
executor/linux/secret.go:202: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 219 in executor/linux/secret.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] executor/linux/secret.go#L219

non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
executor/linux/secret.go:219: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 236 in executor/linux/secret.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] executor/linux/secret.go#L236

non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
Raw output
executor/linux/secret.go:236: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 169 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] cmd/vela-worker/operate.go#L169

branch statements should not be cuddled if block has more than two lines (wsl)
Raw output
cmd/vela-worker/operate.go:169:6: branch statements should not be cuddled if block has more than two lines (wsl)
					continue
					^

Check failure on line 175 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions 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 182 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions github-actions / golangci

[golangci] cmd/vela-worker/operate.go#L182

return statements should not be cuddled if block has more than two lines (wsl)
Raw output
cmd/vela-worker/operate.go:182:6: return statements should not be cuddled if block has more than two lines (wsl)
					return nil
					^

Check failure on line 200 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions 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 204 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions 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 210 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions 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
						^

Check failure on line 121 in cmd/vela-worker/operate.go

See this annotation in the file changed.

@github-actions 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)
					^