enhance(secrets)!: use the same allow_events system as repos for secrets #5257
ecrupper
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
reviewdog.yml
on: pull_request
diff-review
1m 50s
full-review
1m 47s
Annotations
20 errors
|
full-review:
queue/redis/redis_test.go#L95
`_steps` is unused (deadcode)
|
|
full-review:
compiler/registry/github/github.go#L41
string `/api/v3/` has 5 occurrences, make it a constant (goconst)
|
|
full-review:
mock/server/authentication.go#L41
string `error` has 4 occurrences, make it a constant (goconst)
|
|
full-review:
cmd/vela-server/schedule.go#L138
cyclomatic complexity 31 of func `processSchedule` is high (> 30) (gocyclo)
|
|
full-review:
queue/redis/route_test.go#L62
G601: Implicit memory aliasing in for loop. (gosec)
|
|
full-review:
compiler/native/compile_test.go#L1818
G601: Implicit memory aliasing in for loop. (gosec)
|
|
full-review:
compiler/native/compile_test.go#L3129
G601: Implicit memory aliasing in for loop. (gosec)
|
|
full-review:
database/deployment/count_repo.go#L14
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
database/deployment/create.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
database/deployment/list_repo.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
queue/redis/redis_test.go#L95
`_steps` is unused (deadcode)
|
|
diff-review:
compiler/registry/github/github.go#L41
string `/api/v3/` has 5 occurrences, make it a constant (goconst)
|
|
diff-review:
mock/server/authentication.go#L41
string `error` has 4 occurrences, make it a constant (goconst)
|
|
diff-review:
cmd/vela-server/schedule.go#L138
cyclomatic complexity 31 of func `processSchedule` is high (> 30) (gocyclo)
|
|
diff-review:
queue/redis/route_test.go#L62
G601: Implicit memory aliasing in for loop. (gosec)
|
|
diff-review:
compiler/native/compile_test.go#L1818
G601: Implicit memory aliasing in for loop. (gosec)
|
|
diff-review:
compiler/native/compile_test.go#L3129
G601: Implicit memory aliasing in for loop. (gosec)
|
|
diff-review:
database/user/table.go#L48
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
database/user/update.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
database/user/create.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|