Skip to content

fix(deps): update module github.com/google/go-github/v55 to v56 #11352

fix(deps): update module github.com/google/go-github/v55 to v56

fix(deps): update module github.com/google/go-github/v55 to v56 #11352

Triggered via push October 17, 2023 15:36
Status Failure
Total duration 2m 11s
Artifacts

validate.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

62 errors
[golangci] router/middleware/scm_test.go#L12: router/middleware/scm_test.go#L12
could not import github.com/go-vela/server/scm/github (-: # github.com/go-vela/server/scm/github
[golangci] scm/github/repo.go#L558: scm/github/repo.go#L558
cannot use true (untyped bool constant) as int value in argument to client.Repositories.GetBranch) (typecheck)
[golangci] router/middleware/perm/perm_test.go#L22: router/middleware/perm/perm_test.go#L22
could not import github.com/go-vela/server/scm/github (-: # github.com/go-vela/server/scm/github
[golangci] scm/github/repo.go#L558: scm/github/repo.go#L558
cannot use true (untyped bool constant) as int value in argument to client.Repositories.GetBranch) (typecheck)
[golangci] router/middleware/pipeline/pipeline_test.go#L25: router/middleware/pipeline/pipeline_test.go#L25
could not import github.com/go-vela/server/scm/github (-: # github.com/go-vela/server/scm/github
[golangci] scm/github/repo.go#L558: scm/github/repo.go#L558
cannot use true (untyped bool constant) as int value in argument to client.Repositories.GetBranch) (typecheck)
[golangci] scm/github/access.go#L1: scm/github/access.go#L1
: # github.com/go-vela/server/scm/github [github.com/go-vela/server/scm/github.test]
[golangci] scm/github/repo.go#L558: scm/github/repo.go#L558
cannot use true (untyped bool constant) as int value in argument to client.Repositories.GetBranch (typecheck)
[golangci] queue/redis/redis_test.go#L95: queue/redis/redis_test.go#L95
`_steps` is unused (deadcode)
[golangci] database/pipeline/create.go#L3: database/pipeline/create.go#L3
3-53 lines are duplicate of `database/pipeline/update.go:3-54` (dupl)
[golangci] database/pipeline/update.go#L3: database/pipeline/update.go#L3
3-54 lines are duplicate of `database/pipeline/create.go:3-53` (dupl)
[golangci] mock/server/schedule.go#L3: mock/server/schedule.go#L3
3-211 lines are duplicate of `mock/server/secret.go:4-167` (dupl)
[golangci] secret/native/count_test.go#L36: secret/native/count_test.go#L36
Error return value of `db.DeleteSecret` is not checked (errcheck)
[golangci] secret/native/create_test.go#L39: secret/native/create_test.go#L39
Error return value of `db.DeleteSecret` is not checked (errcheck)
[golangci] secret/native/create_test.go#L86: secret/native/create_test.go#L86
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#L115: internal/token/refresh_test.go#L115
Error return value of `db.DeleteUser` is not checked (errcheck)
[golangci] router/middleware/worker/worker_test.go#L60: router/middleware/worker/worker_test.go#L60
Error return value of `db.DeleteWorker` is not checked (errcheck)
[golangci] router/middleware/claims/claims_test.go#L276: router/middleware/claims/claims_test.go#L276
Error return value of `db.DeleteUser` is not checked (errcheck)
[golangci] router/middleware/org/org_test.go#L67: router/middleware/org/org_test.go#L67
Error return value of `db.DeleteRepo` is not checked (errcheck)
[golangci] router/middleware/repo/repo_test.go#L74: router/middleware/repo/repo_test.go#L74
Error return value of `db.DeleteRepo` is not checked (errcheck)
[golangci] router/middleware/build/build_test.go#L91: router/middleware/build/build_test.go#L91
Error return value of `db.DeleteBuild` is not checked (errcheck)
[golangci] router/middleware/build/build_test.go#L92: router/middleware/build/build_test.go#L92
Error return value of `db.DeleteRepo` is not checked (errcheck)
[golangci] router/middleware/service/service_test.go#L80: router/middleware/service/service_test.go#L80
Error return value of `db.DeleteBuild` is not checked (errcheck)
[golangci] router/middleware/service/service_test.go#L82: router/middleware/service/service_test.go#L82
Error return value of `db.DeleteService` is not checked (errcheck)
[golangci] router/middleware/service/service_test.go#L222: router/middleware/service/service_test.go#L222
Error return value of `db.DeleteBuild` is not checked (errcheck)
[golangci] router/middleware/step/step_test.go#L84: router/middleware/step/step_test.go#L84
Error return value of `db.DeleteStep` is not checked (errcheck)
[golangci] compiler/native/expand.go#L48: compiler/native/expand.go#L48
Function 'ExpandSteps' has too many statements (71 > 70) (funlen)
[golangci] mock/server/authentication.go#L41: mock/server/authentication.go#L41
string `error` has 4 occurrences, make it a constant (goconst)
[golangci] database/database.go#L82: database/database.go#L82
Comment should end in a period (godot)
[golangci] database/resource.go#L1: database/resource.go#L1
Actual: Copyright (c) 2023 Target Brands, Ine. All rights reserved.
[golangci] database/validate.go#L1: database/validate.go#L1
Actual: Copyright (c) 2023 Target Brands, Ine. All rights reserved.
[golangci] database/validate_test.go#L1: database/validate_test.go#L1
Actual: Copyright (c) 2023 Target Brands, Ine. All rights reserved.
[golangci] api/queue/queue.go#L1: api/queue/queue.go#L1
Actual: Copyright (c) 2023 Target Brands, Inc. All rights reserved.
[golangci] database/schedule/count.go#L6: database/schedule/count.go#L6
File is not `goimports`-ed (goimports)
[golangci] database/schedule/count_active.go#L6: database/schedule/count_active.go#L6
File is not `goimports`-ed (goimports)
[golangci] database/schedule/count_repo.go#L6: database/schedule/count_repo.go#L6
File is not `goimports`-ed (goimports)
[golangci] queue/redis/route_test.go#L62: queue/redis/route_test.go#L62
G601: Implicit memory aliasing in for loop. (gosec)
[golangci] compiler/native/compile_test.go#L1808: compiler/native/compile_test.go#L1808
G601: Implicit memory aliasing in for loop. (gosec)
[golangci] compiler/native/compile_test.go#L3112: compiler/native/compile_test.go#L3112
G601: Implicit memory aliasing in for loop. (gosec)
[golangci] database/hook/index.go#L19: database/hook/index.go#L19
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/hook/get_webhook.go#L14: database/hook/get_webhook.go#L14
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/hook/last_repo.go#L18: database/hook/last_repo.go#L18
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/repo/repo_test.go#L211: database/repo/repo_test.go#L211
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/pipeline/pipeline_test.go#L207: database/pipeline/pipeline_test.go#L207
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] database/build/build_test.go#L271: database/build/build_test.go#L271
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] compiler/native/parse_test.go#L851: compiler/native/parse_test.go#L851
unused-parameter: parameter 'p' seems to be unused, consider removing or renaming it as _ (revive)
[golangci] api/metrics.go#L444: api/metrics.go#L444
unnecessary leading newline (whitespace)
[golangci] api/schedule/update.go#L128: api/schedule/update.go#L128
if statements should only be cuddled with assignments (wsl)
[golangci] database/resource.go#L30: database/resource.go#L30
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
[golangci] database/resource.go#L41: database/resource.go#L41
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
[golangci] database/resource.go#L54: database/resource.go#L54
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
[golangci] database/schedule/create.go#L3: database/schedule/create.go#L3
directive `//nolint:dupl // ignore similar code with update.go` is unused for linter "dupl" (nolintlint)
[golangci] compiler/native/compile.go#L264: compiler/native/compile.go#L264
directive `//nolint:dupl,lll // linter thinks the steps and stages workflows are identical` is unused for linter "dupl" (nolintlint)
[golangci] compiler/native/compile.go#L361: compiler/native/compile.go#L361
directive `//nolint:dupl,lll // linter thinks the steps and stages workflows are identical` is unused for linter "dupl" (nolintlint)
[golangci] api/admin/step.go#L3: api/admin/step.go#L3
directive `//nolint:dupl // ignore similar code` is unused for linter "dupl" (nolintlint)
[golangci] secret/vault/count.go#L37: secret/vault/count.go#L37
directive `//nolint:staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)
[golangci] secret/vault/get.go#L40: secret/vault/get.go#L40
directive `//nolint:ineffassign,staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)
[golangci] secret/vault/list.go#L45: secret/vault/list.go#L45
directive `//nolint:staticcheck // ignore false positive` is unused for linter "staticcheck" (nolintlint)
validate: scm/github/repo.go#L558
cannot use true (untyped bool constant) as int value in argument to client.Repositories.GetBranch
validate
cannot use true (untyped bool constant) as int value in argument to client.Repositories.GetBranch
validate
Process completed with exit code 1.