Skip to content
reviewdog

test(integration): refactor to run db integration test as matrix #5135

Sign in to view logs

test(integration): refactor to run db integration test as matrix

test(integration): refactor to run db integration test as matrix #5135

Triggered via pull request December 6, 2023 16:05
@wass3rw3rkwass3rw3rk
synchronize #1018
test/matrix_integration
Status Success
Total duration 1m 53s
Artifacts –

reviewdog.yml

on: pull_request
diff-review
1m 40s
diff-review
full-review
1m 29s
full-review
Fit to window
Zoom out
Zoom in

Annotations

20 errors
full-review: queue/redis/redis_test.go#L95
`_steps` is unused (deadcode)
full-review: mock/server/authentication.go#L41
string `error` has 4 occurrences, make it a constant (goconst)
full-review: compiler/registry/github/github.go#L41
string `/api/v3/` has 5 occurrences, make it a constant (goconst)
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: cmd/vela-server/schedule.go#L106
`occurence` is a misspelling of `occurrence` (misspell)
full-review: database/log/update.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/log/count.go#L12
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/log/delete.go#L14
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: 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: cmd/vela-server/schedule.go#L106
`occurence` is a misspelling of `occurrence` (misspell)
diff-review: database/pipeline/get.go#L14
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/pipeline/delete.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/pipeline/table.go#L62
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)