more integration test fixes #13519
ecrupperAnnotations
33 errors
|
[golangci] database/executable/table.go#L38:
database/executable/table.go#L38
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] database/executable/create.go#L17:
database/executable/create.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] database/executable/pop.go#L16:
database/executable/pop.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[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] compiler/template/starlark/starlark.go#L131:
compiler/template/starlark/starlark.go#L131
only one cuddle assignment allowed before if statement (wsl)
|
|
[golangci] database/pipeline/create.go#L3:
database/pipeline/create.go#L3
3-54 lines are duplicate of `database/pipeline/update.go:3-55` (dupl)
|
|
[golangci] database/pipeline/update.go#L3:
database/pipeline/update.go#L3
3-55 lines are duplicate of `database/pipeline/create.go:3-54` (dupl)
|
|
[golangci] database/deployment/create.go#L3:
database/deployment/create.go#L3
3-34 lines are duplicate of `database/deployment/update.go:3-34` (dupl)
|
|
[golangci] database/deployment/update.go#L3:
database/deployment/update.go#L3
3-34 lines are duplicate of `database/deployment/create.go:3-34` (dupl)
|
|
[golangci] mock/server/schedule.go#L3:
mock/server/schedule.go#L3
3-281 lines are duplicate of `mock/server/secret.go:4-181` (dupl)
|
|
[golangci] database/worker/create.go#L3:
database/worker/create.go#L3
3-38 lines are duplicate of `database/worker/update.go:3-38` (dupl)
|
|
[golangci] database/worker/update.go#L3:
database/worker/update.go#L3
3-38 lines are duplicate of `database/worker/create.go:3-38` (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] api/pipeline/validate.go#L71:
api/pipeline/validate.go#L71
71-110 lines are duplicate of `api/pipeline/compile.go:73-112` (dupl)
|
|
[golangci] compiler/native/expand.go#L47:
compiler/native/expand.go#L47
Function 'ExpandSteps' has too many statements (77 > 70) (funlen)
|
|
[golangci] mock/server/authentication.go#L42:
mock/server/authentication.go#L42
string `error` has 4 occurrences, make it a constant (goconst)
|
|
[golangci] queue/redis/route_test.go#L62:
queue/redis/route_test.go#L62
G601: Implicit memory aliasing in for loop. (gosec)
|
|
[golangci] scm/github/authentication.go#L59:
scm/github/authentication.go#L59
unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] router/middleware/logger.go#L43:
router/middleware/logger.go#L43
unused-parameter: parameter 'timeFormat' seems to be unused, consider removing or renaming it as _ (revive)
|
|
[golangci] api/webhook/post.go#L192:
api/webhook/post.go#L192
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
[golangci] api/webhook/post.go#L515:
api/webhook/post.go#L515
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
[golangci] api/webhook/post.go#L526:
api/webhook/post.go#L526
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
[golangci] scm/github/access.go#L34:
scm/github/access.go#L34
Function `newClientToken` should pass the context parameter (contextcheck)
|
|
[golangci] scm/github/access.go#L103:
scm/github/access.go#L103
Function `newClientToken` should pass the context parameter (contextcheck)
|
|
[golangci] scm/github/access.go#L153:
scm/github/access.go#L153
Function `newClientToken` should pass the context parameter (contextcheck)
|
|
[golangci] scm/github/repo.go#L525:
scm/github/repo.go#L525
SA1019: client.Repositories.List is deprecated: Use RepositoriesService.ListByUser or RepositoriesService.ListByAuthenticatedUser instead. (staticcheck)
|
|
[golangci] router/middleware/header.go#L34:
router/middleware/header.go#L34
if statements should only be cuddled with assignments (wsl)
|
|
[golangci] router/middleware/header.go#L38:
router/middleware/header.go#L38
expressions should not be cuddled with blocks (wsl)
|
|
[golangci] secret/vault/get.go#L40:
secret/vault/get.go#L40
directive `//nolint:ineffassign,staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)
|