test

chore: undo comments #10285

Summary
Jobs
- test
- golangci
Run details
- Usage
- Workflow file

Triggered via push July 28, 2023 18:49

jbrockopp

pushed 061ff80

hackathon/integration

Status Success

Total duration 2m 38s

Artifacts –

test.yml

on: push

test

2m 28s

Annotations

47 errors

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

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

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

5-54 lines are duplicate of `database/pipeline/create.go:5-53` (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#L89: secret/native/create_test.go#L89

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#L68: router/middleware/org/org_test.go#L68

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#L75: router/middleware/repo/repo_test.go#L75

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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#L79: database/database.go#L79

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] cmd/vela-server/schedule.go#L106: cmd/vela-server/schedule.go#L106

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

[golangci] database/pipeline/pipeline_test.go#L205: database/pipeline/pipeline_test.go#L205

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/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] 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#L443: api/metrics.go#L443

unnecessary leading newline (whitespace)

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

unnecessary trailing newline (whitespace)

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

expressions should not be cuddled with blocks (wsl)

[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#L303: api/build/create.go#L303

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

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

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: undo comments #10285

Summary