validate

with encryption resource opt #10497

Triggered via push August 22, 2023 20:21

ecrupper

pushed 3ed0a6d

pipeline-build-table

Status Success

Total duration 2m 4s

Artifacts –

validate.yml

on: push

validate

1m 51s

Annotations

60 errors

[golangci] queue/redis/redis_test.go#L95: queue/redis/redis_test.go#L95

`_steps` is unused (deadcode)

[golangci] database/pipeline/create.go#L5: database/pipeline/create.go#L5

5-55 lines are duplicate of `database/pipeline/update.go:5-56` (dupl)

[golangci] database/pipeline/update.go#L5: database/pipeline/update.go#L5

5-56 lines are duplicate of `database/pipeline/create.go:5-55` (dupl)

[golangci] mock/server/schedule.go#L5: mock/server/schedule.go#L5

5-213 lines are duplicate of `mock/server/secret.go:6-169` (dupl)

[golangci] secret/native/count_test.go#L37: secret/native/count_test.go#L37

Error return value of `db.DeleteSecret` is not checked (errcheck)

[golangci] secret/native/create_test.go#L40: secret/native/create_test.go#L40

Error return value of `db.DeleteSecret` is not checked (errcheck)

[golangci] secret/native/create_test.go#L87: secret/native/create_test.go#L87

Error return value of `db.DeleteSecret` is not checked (errcheck)

[golangci] internal/token/refresh_test.go#L54: internal/token/refresh_test.go#L54

Error return value of `db.DeleteUser` is not checked (errcheck)

[golangci] internal/token/refresh_test.go#L113: internal/token/refresh_test.go#L113

Error return value of `db.DeleteUser` is not checked (errcheck)

[golangci] router/middleware/worker/worker_test.go#L61: router/middleware/worker/worker_test.go#L61

Error return value of `db.DeleteWorker` is not checked (errcheck)

[golangci] router/middleware/org/org_test.go#L69: router/middleware/org/org_test.go#L69

Error return value of `db.DeleteRepo` is not checked (errcheck)

[golangci] router/middleware/claims/claims_test.go#L277: router/middleware/claims/claims_test.go#L277

Error return value of `db.DeleteUser` is not checked (errcheck)

[golangci] router/middleware/repo/repo_test.go#L76: router/middleware/repo/repo_test.go#L76

Error return value of `db.DeleteRepo` is not checked (errcheck)

[golangci] router/middleware/pipeline/pipeline_test.go#L106: router/middleware/pipeline/pipeline_test.go#L106

Error return value of `db.DeletePipeline` is not checked (errcheck)

[golangci] router/middleware/pipeline/pipeline_test.go#L107: router/middleware/pipeline/pipeline_test.go#L107

Error return value of `db.DeleteRepo` is not checked (errcheck)

[golangci] router/middleware/build/build_test.go#L93: router/middleware/build/build_test.go#L93

Error return value of `db.DeleteBuild` is not checked (errcheck)

[golangci] router/middleware/perm/perm_test.go#L448: router/middleware/perm/perm_test.go#L448

Error return value of `db.DeleteBuild` is not checked (errcheck)

[golangci] router/middleware/perm/perm_test.go#L539: router/middleware/perm/perm_test.go#L539

Error return value of `db.DeleteBuild` is not checked (errcheck)

[golangci] router/middleware/service/service_test.go#L84: router/middleware/service/service_test.go#L84

Error return value of `db.DeleteService` is not checked (errcheck)

[golangci] router/middleware/step/step_test.go#L86: router/middleware/step/step_test.go#L86

Error return value of `db.DeleteStep` is not checked (errcheck)

[golangci] cmd/vela-server/server.go#L28: cmd/vela-server/server.go#L28

Function 'server' is too long (184 > 160) (funlen)

[golangci] mock/server/authentication.go#L43: mock/server/authentication.go#L43

string `error` has 3 occurrences, make it a constant (goconst)

[golangci] database/database.go#L84: database/database.go#L84

Comment should end in a period (godot)

[golangci] random/random.go#L1: random/random.go#L1

[golangci] version/version.go#L1: version/version.go#L1

[golangci] compiler/registry/doc.go#L1: compiler/registry/doc.go#L1

[golangci] database/resource.go#L1: database/resource.go#L1

[golangci] database/validate.go#L1: database/validate.go#L1

[golangci] database/validate_test.go#L1: database/validate_test.go#L1

[golangci] compiler/template/starlark/render.go#L10: compiler/template/starlark/render.go#L10

File is not `goimports`-ed (goimports)

[golangci] database/schedule/count.go#L8: database/schedule/count.go#L8

File is not `goimports`-ed (goimports)

[golangci] database/schedule/count_active.go#L8: database/schedule/count_active.go#L8

File is not `goimports`-ed (goimports)

[golangci] queue/redis/route_test.go#L64: queue/redis/route_test.go#L64

G601: Implicit memory aliasing in for loop. (gosec)

[golangci] compiler/native/compile_test.go#L1795: compiler/native/compile_test.go#L1795

G601: Implicit memory aliasing in for loop. (gosec)

[golangci] compiler/native/compile_test.go#L3092: compiler/native/compile_test.go#L3092

G601: Implicit memory aliasing in for loop. (gosec)

[golangci] cmd/vela-server/schedule.go#L107: cmd/vela-server/schedule.go#L107

`occurence` is a misspelling of `occurrence` (misspell)

[golangci] database/schedule/count.go#L13: database/schedule/count.go#L13

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] database/schedule/get.go#L15: database/schedule/get.go#L15

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] database/schedule/table.go#L53: database/schedule/table.go#L53

unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] database/build/build_test.go#L273: database/build/build_test.go#L273

unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] database/user/user_test.go#L198: database/user/user_test.go#L198

unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] database/repo/repo_test.go#L213: database/repo/repo_test.go#L213

unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] scm/github/authentication.go#L60: scm/github/authentication.go#L60

unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] compiler/native/parse_test.go#L849: compiler/native/parse_test.go#L849

unused-parameter: parameter 'p' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] router/middleware/logger.go#L30: router/middleware/logger.go#L30

unused-parameter: parameter 'timeFormat' seems to be unused, consider removing or renaming it as _ (revive)

[golangci] api/metrics.go#L446: api/metrics.go#L446

unnecessary leading newline (whitespace)

[golangci] router/router.go#L139: router/router.go#L139

unnecessary trailing newline (whitespace)

[golangci] cmd/vela-server/schedule.go#L276: cmd/vela-server/schedule.go#L276

expressions should not be cuddled with blocks (wsl)

[golangci] api/webhook/post.go#L720: api/webhook/post.go#L720

Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)

[golangci] api/webhook/post.go#L731: api/webhook/post.go#L731

Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)

[golangci] api/webhook/post.go#L764: api/webhook/post.go#L764

Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)

[golangci] database/schedule/create.go#L5: database/schedule/create.go#L5

directive `//nolint:dupl // ignore similar code with update.go` is unused for linter "dupl" (nolintlint)

[golangci] compiler/native/compile.go#L266: compiler/native/compile.go#L266

directive `//nolint:dupl,lll // linter thinks the steps and stages workflows are identical` is unused for linter "dupl" (nolintlint)

[golangci] compiler/native/compile.go#L363: compiler/native/compile.go#L363

directive `//nolint:dupl,lll // linter thinks the steps and stages workflows are identical` is unused for linter "dupl" (nolintlint)

[golangci] secret/vault/count.go#L38: secret/vault/count.go#L38

directive `//nolint:staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)

[golangci] secret/vault/get.go#L41: secret/vault/get.go#L41

directive `//nolint:ineffassign,staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)

[golangci] secret/vault/list.go#L46: secret/vault/list.go#L46

directive `//nolint:staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)

[golangci] api/build/create.go#L304: api/build/create.go#L304

directive `//nolint:dupl // ignore duplicate code` is unused for linter "dupl" (nolintlint)

[golangci] api/build/restart.go#L295: api/build/restart.go#L295

directive `//nolint:dupl // ignore duplicate code` is unused for linter "dupl" (nolintlint)

[golangci] api/secret/create.go#L81: api/secret/create.go#L81

directive `//nolint:funlen // suppress long function error` is unused for linter "funlen" (nolintlint)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

with encryption resource opt #10497

Summary