Merge branch 'main' into refactor/nested-api/pipeline #14521
ecrupperAnnotations
46 errors
|
[golangci] api/webhook/post.go#L193:
api/webhook/post.go#L193
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
[golangci] api/webhook/post.go#L662:
api/webhook/post.go#L662
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
[golangci] api/webhook/post.go#L673:
api/webhook/post.go#L673
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
[golangci] database/executable/clean_test.go#L130:
database/executable/clean_test.go#L130
SA1019: library.Build is deprecated: use Build from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
[golangci] database/executable/clean_test.go#L131:
database/executable/clean_test.go#L131
SA1019: library.Build is deprecated: use Build from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
[golangci] internal/token/generate_rsa.go#L19:
internal/token/generate_rsa.go#L19
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] scm/github/authentication.go#L36:
scm/github/authentication.go#L36
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] scm/github/authentication.go#L133:
scm/github/authentication.go#L133
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] mock/server/schedule.go#L3:
mock/server/schedule.go#L3
3-284 lines are duplicate of `mock/server/secret.go:4-181` (dupl)
|
|
[golangci] mock/server/authentication.go#L84:
mock/server/authentication.go#L84
string `error` has 4 occurrences, make it a constant (goconst)
|
|
[golangci] compiler/types/yaml/stage.go#L91:
compiler/types/yaml/stage.go#L91
return statements should not be cuddled if block has more than two lines (wsl)
|
|
[golangci] compiler/types/yaml/stage.go#L97:
compiler/types/yaml/stage.go#L97
return statements should not be cuddled if block has more than two lines (wsl)
|
|
[golangci] compiler/template/starlark/render.go#L44:
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)
|
|
[golangci] compiler/template/starlark/render.go#L152:
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)
|
|
[golangci] compiler/template/starlark/starlark.go#L131:
compiler/template/starlark/starlark.go#L131
only one cuddle assignment allowed before if statement (wsl)
|
|
[golangci] database/types/build.go#L384:
database/types/build.go#L384
G115: integer overflow conversion int -> int32 (gosec)
|
|
[golangci] database/types/build.go#L385:
database/types/build.go#L385
G115: integer overflow conversion int -> int32 (gosec)
|
|
[golangci] database/types/repo.go#L336:
database/types/repo.go#L336
G115: integer overflow conversion int -> int32 (gosec)
|
|
[golangci] database/types/settings.go#L134:
database/types/settings.go#L134
G115: integer overflow conversion int64 -> uint64 (gosec)
|
|
[golangci] database/types/settings.go#L205:
database/types/settings.go#L205
G115: integer overflow conversion uint64 -> int64 (gosec)
|
|
[golangci] secret/vault/refresh_test.go#L45:
secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] secret/vault/refresh_test.go#L71:
secret/vault/refresh_test.go#L71
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] secret/vault/refresh_test.go#L108:
secret/vault/refresh_test.go#L108
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] secret/vault/refresh_test.go#L128:
secret/vault/refresh_test.go#L128
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] secret/vault/refresh_test.go#L201:
secret/vault/refresh_test.go#L201
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] api/pipeline/validate.go#L78:
api/pipeline/validate.go#L78
78-110 lines are duplicate of `api/pipeline/compile.go:80-112` (dupl)
|
|
[golangci] api/pipeline/validate.go#L3:
api/pipeline/validate.go#L3
3-110 lines are duplicate of `api/pipeline/expand.go:4-112` (dupl)
|
|
[golangci] database/deployment/create.go#L3:
database/deployment/create.go#L3
3-37 lines are duplicate of `database/deployment/update.go:3-37` (dupl)
|
|
[golangci] database/deployment/update.go#L3:
database/deployment/update.go#L3
3-37 lines are duplicate of `database/deployment/create.go:3-37` (dupl)
|
|
[golangci] database/service/create.go#L3:
database/service/create.go#L3
3-41 lines are duplicate of `database/service/update.go:3-41` (dupl)
|
|
[golangci] database/service/update.go#L3:
database/service/update.go#L3
3-41 lines are duplicate of `database/service/create.go:3-41` (dupl)
|
|
[golangci] compiler/types/pipeline/ruleset.go#L134:
compiler/types/pipeline/ruleset.go#L134
cyclomatic complexity 31 of func `(*Rules).Match` is high (> 30) (gocyclo)
|
|
[golangci] database/hook/get.go#L3:
database/hook/get.go#L3
3-40 lines are duplicate of `database/hook/get_webhook.go:3-40` (dupl)
|
|
[golangci] database/hook/get_webhook.go#L3:
database/hook/get_webhook.go#L3
3-40 lines are duplicate of `database/hook/get.go:3-40` (dupl)
|
|
[golangci] database/hook/create.go#L3:
database/hook/create.go#L3
3-39 lines are duplicate of `database/hook/update.go:3-42` (dupl)
|
|
[golangci] database/hook/update.go#L3:
database/hook/update.go#L3
3-42 lines are duplicate of `database/hook/create.go:3-39` (dupl)
|
|
[golangci] database/step/create.go#L3:
database/step/create.go#L3
3-41 lines are duplicate of `database/step/update.go:3-41` (dupl)
|
|
[golangci] database/step/update.go#L3:
database/step/update.go#L3
3-41 lines are duplicate of `database/step/create.go:3-41` (dupl)
|
|
[golangci] database/worker/create.go#L3:
database/worker/create.go#L3
3-41 lines are duplicate of `database/worker/update.go:3-41` (dupl)
|
|
[golangci] database/worker/update.go#L3:
database/worker/update.go#L3
3-41 lines are duplicate of `database/worker/create.go:3-41` (dupl)
|
|
[golangci] database/pipeline/create.go#L3:
database/pipeline/create.go#L3
3-51 lines are duplicate of `database/pipeline/update.go:3-52` (dupl)
|
|
[golangci] database/pipeline/update.go#L3:
database/pipeline/update.go#L3
3-52 lines are duplicate of `database/pipeline/create.go:3-51` (dupl)
|
|
[golangci] router/middleware/logger.go#L48:
router/middleware/logger.go#L48
unused-parameter: parameter 'timeFormat' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] compiler/native/compile.go#L332:
compiler/native/compile.go#L332
Function `modifyConfig` should pass the context parameter (contextcheck)
|
|
[golangci] compiler/native/compile.go#L427:
compiler/native/compile.go#L427
Function `modifyConfig` should pass the context parameter (contextcheck)
|
|
[golangci] secret/vault/get.go#L40:
secret/vault/get.go#L40
directive `//nolint:ineffassign,staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)
|