Skip to content
reviewdog

refactor(pipeline): use server API types for pipeline and migrate compiler types #6347

Sign in to view logs

refactor(pipeline): use server API types for pipeline and migrate compiler types

refactor(pipeline): use server API types for pipeline and migrate compiler types #6347

Triggered via pull request October 3, 2024 17:08
@ecrupperecrupper
opened #1200
refactor/nested-api/pipeline
Status Failure
Total duration 2m 7s
Artifacts –

reviewdog.yml

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

Annotations

20 errors
full-review: mock/server/authentication.go#L84
string `error` has 4 occurrences, make it a constant (goconst)
full-review: compiler/types/pipeline/ruleset.go#L134
cyclomatic complexity 31 of func `(*Rules).Match` is high (> 30) (gocyclo)
full-review: database/types/build.go#L384
G115: integer overflow conversion int -> int32 (gosec)
full-review: database/types/build.go#L385
G115: integer overflow conversion int -> int32 (gosec)
full-review: database/types/repo.go#L336
G115: integer overflow conversion int -> int32 (gosec)
full-review: database/types/settings.go#L134
G115: integer overflow conversion int64 -> uint64 (gosec)
full-review: database/types/settings.go#L205
G115: integer overflow conversion uint64 -> int64 (gosec)
full-review: secret/vault/delete.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
full-review: secret/vault/refresh_test.go#L71
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: mock/server/authentication.go#L84
string `error` has 4 occurrences, make it a constant (goconst)
diff-review: compiler/types/pipeline/ruleset.go#L134
cyclomatic complexity 31 of func `(*Rules).Match` is high (> 30) (gocyclo)
diff-review: database/types/build.go#L384
G115: integer overflow conversion int -> int32 (gosec)
diff-review: database/types/build.go#L385
G115: integer overflow conversion int -> int32 (gosec)
diff-review: database/types/repo.go#L336
G115: integer overflow conversion int -> int32 (gosec)
diff-review: database/types/settings.go#L134
G115: integer overflow conversion int64 -> uint64 (gosec)
diff-review: database/types/settings.go#L205
G115: integer overflow conversion uint64 -> int64 (gosec)
diff-review: secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: secret/vault/refresh_test.go#L71
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: secret/vault/refresh_test.go#L108
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)