chore(deps): update codecov/codecov-action action to v5 #1980
renovate[bot]reviewdog.yml
on: pull_request
diff-review
1m 47s
full-review
1m 48s
Annotations
20 errors and 4 warnings
|
diff-review:
internal/context/context_test.go#L132
if statements should only be cuddled with assignments (wsl)
|
|
diff-review:
internal/context/context_test.go#L148
assignments should only be cuddled with other assignments (wsl)
|
|
diff-review:
internal/context/context_test.go#L149
only one cuddle assignment allowed before if statement (wsl)
|
|
diff-review:
internal/service/snapshot_test.go#L137
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
internal/service/upload_test.go#L165
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
runtime/kubernetes/mock.go#L143
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
|
|
diff-review:
runtime/kubernetes/pod_tracker.go#L226
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
|
|
diff-review:
cmd/vela-worker/exec.go#L29
cyclomatic complexity 31 of func `(*Worker).exec` is high (> 30) (gocyclo)
|
|
diff-review:
mock/docker/plugin.go#L22
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
mock/docker/plugin.go#L30
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
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:
internal/service/snapshot_test.go#L137
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
internal/service/upload_test.go#L165
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
runtime/kubernetes/mock.go#L143
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
|
|
full-review:
runtime/kubernetes/pod_tracker.go#L226
Error return value of `(k8s.io/client-go/tools/cache.SharedInformer).AddEventHandler` is not checked (errcheck)
|
|
full-review:
cmd/vela-worker/exec.go#L29
cyclomatic complexity 31 of func `(*Worker).exec` is high (> 30) (gocyclo)
|
|
full-review:
mock/docker/mock.go#L58
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
mock/docker/container.go#L35
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
diff-review
Input 'fail_on_error' has been deprecated with message: Deprecated, use `fail_level` instead.
|
|
full-review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
full-review
Input 'fail_on_error' has been deprecated with message: Deprecated, use `fail_level` instead.
|