Report for github.com/roistat/go-clickhouse
A+ Excellent! Found 9 issues across 14 files
gofmt92%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
-
-
go-clickhouse/marshal_test.go
- Line 1: warning: file is not gofmted with -s (gofmt)
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
gocyclo92%
Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.
-
-
go-clickhouse/marshal.go
- Line 35: warning: cyclomatic complexity 28 of function unmarshal() is high (> 15) (gocyclo)
golint57%
Golint is a linter for Go source code.
-
-
go-clickhouse/query.go
- Line 8: warning: exported type External should have comment or be unexported (golint)
- Line 14: warning: exported type Func should have comment or be unexported (golint)
- Line 19: warning: exported type Query should have comment or be unexported (golint)
- Line 25: warning: exported method Query.AddExternal should have comment or be unexported (golint)
- Line 29: warning: exported method Query.Iter should have comment or be unexported (golint)
- Line 46: warning: exported method Query.Exec should have comment or be unexported (golint)
- Line 58: warning: exported type Iter should have comment or be unexported (golint)
- Line 67: warning: exported method Iter.Scan should have comment or be unexported (golint)
-
-
go-clickhouse/transport.go
- Line 16: warning: exported type Transport should have comment or be unexported (golint)
- Line 20: warning: exported type HttpTransport should have comment or be unexported (golint)
- Line 24: warning: exported method HttpTransport.Exec should have comment or be unexported (golint)
-
-
go-clickhouse/cluster.go
- Line 8: warning: exported type PingErrorFunc should have comment or be unexported (golint)
- Line 10: warning: exported type Cluster should have comment or be unexported (golint)
- Line 17: warning: exported function NewCluster should have comment or be unexported (golint)
- Line 23: warning: exported method Cluster.IsDown should have comment or be unexported (golint)
- Line 29: warning: exported method Cluster.OnCheckError should have comment or be unexported (golint)
- Line 33: warning: exported method Cluster.ActiveConn should have comment or be unexported (golint)
- Line 43: warning: exported method Cluster.Check should have comment or be unexported (golint)
-
-
go-clickhouse/conn.go
- Line 12: warning: exported type Conn should have comment or be unexported (golint)
- Line 17: warning: exported method Conn.Ping should have comment or be unexported (golint)
-
-
go-clickhouse/errors.go
- Line 9: warning: exported type DbError should have comment or be unexported (golint)
- Line 15: warning: exported method DbError.Code should have comment or be unexported (golint)
- Line 19: warning: exported method DbError.Message should have comment or be unexported (golint)
- Line 23: warning: exported method DbError.Response should have comment or be unexported (golint)
-
-
go-clickhouse/helpers.go
- Line 10: warning: exported type Column should have comment or be unexported (golint)
- Line 11: warning: exported type Columns should have comment or be unexported (golint)
- Line 12: warning: exported type Row should have comment or be unexported (golint)
- Line 13: warning: exported type Rows should have comment or be unexported (golint)
- Line 14: warning: exported type Array should have comment or be unexported (golint)
- Line 17: warning: exported function NewHttpTransport should have comment or be unexported (golint)
- Line 21: warning: exported function NewConn should have comment or be unexported (golint)
- Line 33: warning: exported function NewQuery should have comment or be unexported (golint)
- Line 40: warning: exported function BuildInsert should have comment or be unexported (golint)
- Line 44: warning: exported function BuildMultiInsert should have comment or be unexported (golint)
ineffassign85%
IneffAssign detects ineffectual assignments in Go code.
-
-
go-clickhouse/transport_test.go
- Line 49: warning: ineffectual assignment to query (ineffassign)
-
-
go-clickhouse/transport.go
- Line 55: warning: ineffectual assignment to err (ineffassign)