Add linting, unit tests and acceptance test workflows #4
pskrbasuAnnotations
10 errors and 1 warning
|
Test Linting:
cmd/connect.go#L51
use of `fmt.Println` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
Test Linting:
cmd/plugin.go#L307
use of `fmt.Println` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
Test Linting:
cmd/plugin.go#L364
use of `fmt.Println` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
Test Linting:
cmd/plugin.go#L721
use of `fmt.Printf` forbidden by pattern `^(fmt\.Print(|f|ln)|print|println)$` (forbidigo)
|
|
Test Linting:
internal/collection_state/repository.go#L143
exitAfterDefer: log.Fatalf will exit, and `defer rows.Close()` will not run (gocritic)
|
|
Test Linting:
internal/plugin_manager/plugin_manager.go#L131
G404: Use of weak random number generator (math/rand or math/rand/v2 instead of crypto/rand) (gosec)
|
|
Test Linting:
cmd/collect.go#L142
SA4009: argument tablePattern is overwritten before first use (staticcheck)
|
|
Test Linting:
internal/query/execute.go#L88
SA9003: empty branch (staticcheck)
|
|
Test Linting:
internal/config/partition.go#L64
directive `//nolint: unused` should provide explanation such as `//nolint: unused // this is why` (nolintlint)
|
|
Test Linting:
internal/database/tables.go#L81
directive `//nolint: gosec` should provide explanation such as `//nolint: gosec // this is why` (nolintlint)
|
|
Test Linting
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|