fix(deps): update all non-major dependencies #6389
renovate[bot]reviewdog.yml
on: pull_request
diff-review
1m 21s
full-review
1m 9s
Annotations
20 errors
|
full-review:
compiler/native/compile.go#L332
Function `modifyConfig` should pass the context parameter (contextcheck)
|
|
full-review:
compiler/native/compile.go#L427
Function `modifyConfig` should pass the context parameter (contextcheck)
|
|
full-review:
compiler/types/pipeline/ruleset.go#L134
cyclomatic complexity 31 of func `(*Rules).Match` is high (> 30) (gocyclo)
|
|
full-review:
compiler/types/yaml/stage.go#L91
return statements should not be cuddled if block has more than two lines (wsl)
|
|
full-review:
compiler/types/yaml/stage.go#L97
return statements should not be cuddled if block has more than two lines (wsl)
|
|
full-review:
mock/server/authentication.go#L82
string `error` has 4 occurrences, make it a constant (goconst)
|
|
full-review:
compiler/template/starlark/render.go#L44
SA1019: starlark.ExecFile is deprecated: use [ExecFileOptions] with [syntax.FileOptions] instead, because this function relies on legacy global variables. (staticcheck)
|
|
full-review:
compiler/template/starlark/render.go#L152
SA1019: starlark.ExecFile is deprecated: use [ExecFileOptions] with [syntax.FileOptions] instead, because this function relies on legacy global variables. (staticcheck)
|
|
full-review:
compiler/template/starlark/starlark.go#L131
only one cuddle assignment allowed before if statement (wsl)
|
|
full-review:
scm/github/webhook.go#L86
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
compiler/template/starlark/render.go#L44
SA1019: starlark.ExecFile is deprecated: use [ExecFileOptions] with [syntax.FileOptions] instead, because this function relies on legacy global variables. (staticcheck)
|
|
diff-review:
compiler/template/starlark/render.go#L152
SA1019: starlark.ExecFile is deprecated: use [ExecFileOptions] with [syntax.FileOptions] instead, because this function relies on legacy global variables. (staticcheck)
|
|
diff-review:
compiler/template/starlark/starlark.go#L131
only one cuddle assignment allowed before if statement (wsl)
|
|
diff-review:
compiler/native/compile.go#L332
Function `modifyConfig` should pass the context parameter (contextcheck)
|
|
diff-review:
compiler/native/compile.go#L427
Function `modifyConfig` should pass the context parameter (contextcheck)
|
|
diff-review:
compiler/types/pipeline/ruleset.go#L134
cyclomatic complexity 31 of func `(*Rules).Match` is high (> 30) (gocyclo)
|
|
diff-review:
mock/server/authentication.go#L82
string `error` has 4 occurrences, make it a constant (goconst)
|
|
diff-review:
api/types/build.go#L60
string `...` has 3 occurrences, make it a constant (goconst)
|
|
diff-review:
compiler/registry/github/template.go#L39
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
diff-review:
scm/github/webhook.go#L86
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|