Skip to content

setup golangci-lint and fix the suggestions #585

setup golangci-lint and fix the suggestions

setup golangci-lint and fix the suggestions #585

Triggered via pull request July 8, 2023 18:27
@defbindefbin
synchronize #852
ci-lint
Status Success
Total duration 1m 47s
Artifacts

reviewdog.yml

on: pull_request
runner / go-test
49s
runner / go-test
runner / shellcheck
12s
runner / shellcheck
runner / misspell
18s
runner / misspell
runner / alex
52s
runner / alex
runner / golangci-lint
1m 35s
runner / golangci-lint
runner / gofmt
32s
runner / gofmt
runner / shfmt
12s
runner / shfmt
Fit to window
Zoom out
Zoom in

Annotations

10 errors
runner / golangci-lint: pbm/config.go#L39
string `***` has 13 occurrences, make it a constant (goconst)
runner / golangci-lint: pbm/oplog/restore.go#L350
string `applyOps` has 3 occurrences, make it a constant (goconst)
runner / golangci-lint: pbm/oplog/restore.go#L647
string `createIndexes` has 3 occurrences, make it a constant (goconst)
runner / golangci-lint: cli/delete.go#L29
string `YES` has 3 occurrences, make it a constant (goconst)
runner / golangci-lint: cli/delete.go#L287
string `, base` has 3 occurrences, make it a constant (goconst)
runner / golangci-lint: cli/delete.go#L285
string `, selective` has 5 occurrences, make it a constant (goconst)
runner / golangci-lint: e2e-tests/pkg/tests/sharded/test_delete_backup.go#L48
exitAfterDefer: log.Fatalf will exit, and `defer c.pitrOff()` will not run (gocritic)
runner / golangci-lint: e2e-tests/pkg/tests/sharded/test_dist_commit.go#L53
exitAfterDefer: log.Fatalf will exit, and `defer cur.Close(ctx)` will not run (gocritic)
runner / golangci-lint: e2e-tests/pkg/tests/sharded/test_pitr_basic.go#L74
exitAfterDefer: log.Fatalf will exit, and `defer c.pitrOff()` will not run (gocritic)
runner / golangci-lint: e2e-tests/pkg/tests/sharded/test_pitr_basic.go#L145
exitAfterDefer: log.Fatalln will exit, and `defer tk.Stop()` will not run (gocritic)