Skip to content
reviewdog

refactor: use builtin min/max funcs #6144

Sign in to view logs

refactor: use builtin min/max funcs

refactor: use builtin min/max funcs #6144

Triggered via pull request August 20, 2024 14:58
@wass3rw3rkwass3rw3rk
synchronize #1148
refactor/use-go-minmax
Status Success
Total duration 2m 33s
Artifacts –

reviewdog.yml

on: pull_request
diff-review
2m 21s
diff-review
full-review
2m 7s
full-review
Fit to window
Zoom out
Zoom in

Annotations

20 errors and 2 warnings
full-review: mock/server/authentication.go#L81
string `error` has 4 occurrences, make it a constant (goconst)
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/pop.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
full-review: database/executable/table.go#L38
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)
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)
full-review: secret/vault/refresh_test.go#L201
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
full-review: router/middleware/logger.go#L48
unused-parameter: parameter 'timeFormat' 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: database/executable/table.go#L38
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/executable/pop.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: database/executable/create.go#L17
unused-parameter: parameter 'ctx' 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)
diff-review: secret/vault/refresh_test.go#L201
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
diff-review: router/middleware/logger.go#L48
unused-parameter: parameter 'timeFormat' seems to be unused, consider removing or renaming it as _ (revive)
full-review
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
diff-review
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2