Skip to content

chore: remove legacy event code from repo and secret #5624

chore: remove legacy event code from repo and secret

chore: remove legacy event code from repo and secret #5624

Triggered via pull request April 1, 2024 14:37
Status Failure
Total duration 2m 8s
Artifacts

reviewdog.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

20 errors
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: database/repo/get_org.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/repo/count.go#L12
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/repo/count_user.go#L14
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: scm/github/authentication.go#L58
unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: compiler/native/parse_test.go#L861
unused-parameter: parameter 'p' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
full-review: mock/server/authentication.go#L41
string `error` has 4 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: database/pipeline/count_repo.go#L14
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/pipeline/count.go#L12
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/pipeline/delete.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: scm/github/authentication.go#L58
unused-parameter: parameter 'w' seems to be unused, consider removing or renaming it as _ (revive)
full-review: compiler/native/parse_test.go#L861
unused-parameter: parameter 'p' 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)