Skip to content
reviewdog

feat(db/log): use logrus as logger for db #5972

Sign in to view logs

feat(db/log): use logrus as logger for db

feat(db/log): use logrus as logger for db #5972

Triggered via pull request June 5, 2024 16:31
@wass3rwass3r
opened #1135
feat/db-logging
Status Success
Total duration 2m 8s
Artifacts –

reviewdog.yml

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

Annotations

20 errors
full-review: mock/server/authentication.go#L81
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: database/executable/clean.go#L31
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/executable/create.go#L17
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/executable/pop.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: scm/github/authentication.go#L59
unused-parameter: parameter 'w' 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)
full-review: secret/vault/refresh_test.go#L108
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
full-review: secret/vault/refresh_test.go#L128
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: mock/server/authentication.go#L81
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: database/step/count_build.go#L15
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/step/delete.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/step/list_status.go#L13
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: scm/github/authentication.go#L59
unused-parameter: parameter 'w' 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)
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)
diff-review: secret/vault/refresh_test.go#L128
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)