chore(deps): upgrade docker pkg and fix mock image service #1902
reviewdog.yml
on: pull_request
diff-review
53s
full-review
47s
Annotations
20 errors
full-review:
internal/context/context_test.go#L132
if statements should only be cuddled with assignments (wsl)
|
full-review:
internal/context/context_test.go#L148
assignments should only be cuddled with other assignments (wsl)
|
full-review:
internal/context/context_test.go#L149
only one cuddle assignment allowed before if statement (wsl)
|
full-review:
mock/worker/executor.go#L38
SA1019: library.Executor is deprecated: use Executor from github.com/go-vela/server/api/types instead. (staticcheck)
|
full-review:
mock/worker/executor.go#L58
SA1019: library.Executor is deprecated: use Executor from github.com/go-vela/server/api/types instead. (staticcheck)
|
full-review:
runtime/docker/build.go#L13
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
full-review:
runtime/docker/build.go#L21
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
full-review:
runtime/docker/build.go#L29
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
full-review:
runtime/docker/container.go#L303
SA1019: types.ExecConfig is deprecated: use [container.ExecOptions]. (staticcheck)
|
full-review:
runtime/docker/container.go#L315
SA1019: types.ExecStartCheck is deprecated: use [container.ExecStartOptions] or [container.ExecAttachOptions]. (staticcheck)
|
diff-review:
mock/worker/executor.go#L38
SA1019: library.Executor is deprecated: use Executor from github.com/go-vela/server/api/types instead. (staticcheck)
|
diff-review:
mock/worker/executor.go#L58
SA1019: library.Executor is deprecated: use Executor from github.com/go-vela/server/api/types instead. (staticcheck)
|
diff-review:
runtime/docker/build.go#L13
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
runtime/docker/build.go#L21
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
runtime/docker/build.go#L29
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
runtime/docker/container.go#L303
SA1019: types.ExecConfig is deprecated: use [container.ExecOptions]. (staticcheck)
|
diff-review:
runtime/docker/container.go#L315
SA1019: types.ExecStartCheck is deprecated: use [container.ExecStartOptions] or [container.ExecAttachOptions]. (staticcheck)
|
diff-review:
runtime/docker/network.go#L23
SA1019: types.NetworkCreate is deprecated: use [network.CreateOptions]. (staticcheck)
|
diff-review:
runtime/docker/network.go#L45
SA1019: types.NetworkInspectOptions is deprecated: use [network.InspectOptions]. (staticcheck)
|
diff-review:
mock/docker/container.go#L127
SA1019: types.ContainerExecInspect is deprecated: use [container.ExecInspect]. (staticcheck)
|