Skip to content

try this

try this #10495

Triggered via push August 22, 2023 19:43
Status Success
Total duration 1m 38s
Artifacts

build.yml

on: push
Fit to window
Zoom out
Zoom in

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/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/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/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/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/build/build_test.go#L94: router/middleware/build/build_test.go#L94
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/step/step_test.go#L84: router/middleware/step/step_test.go#L84
Error return value of `db.DeleteBuild` 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] router/middleware/step/step_test.go#L226: router/middleware/step/step_test.go#L226
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] 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
Expected:2023, Actual: 2022 Target Brands, Inc. All rights reserved. (goheader)
[golangci] version/version.go#L1: version/version.go#L1
Expected:2023, Actual: 2022 Target Brands, Inc. All rights reserved. (goheader)
[golangci] compiler/registry/doc.go#L1: compiler/registry/doc.go#L1
Expected:2023, Actual: 2022 Target Brands, Inc. All rights reserved. (goheader)
[golangci] database/resource.go#L1: database/resource.go#L1
Actual: e. All rights reserved.
[golangci] database/validate.go#L1: database/validate.go#L1
Actual: e. All rights reserved.
[golangci] database/validate_test.go#L1: database/validate_test.go#L1
Actual: e. All rights reserved.
[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/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/schedule/update.go#L17: database/schedule/update.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/schedule/delete.go#L16: database/schedule/delete.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/pipeline/pipeline_test.go#L209: database/pipeline/pipeline_test.go#L209
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/log/log_test.go#L179: database/log/log_test.go#L179
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] 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/secret/create.go#L81: api/secret/create.go#L81
directive `//nolint:funlen // suppress long function error` is unused for linter "funlen" (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)