mirror of
https://github.com/hanzo-ds/native.git
synced 2026-10-30 20:00:34 +00:00
zeekay
ef352c1051
proto.Name is sent in ClientHello and lands in the server's system.query_log, so it is wire visible; it now reads datastore/native and is covered by a test. Verified against the live server: the handshake succeeds and the new name appears in query_log. goimports' local-prefixes still named the upstream module, so it was not grouping this module's own imports. Doc comments, the deprecated-error markers in proto/error_codes.go and the package prose are swept. Dead references are removed rather than repointed: go-faster.org/docs/clickhouse and the mangled Datastore/clickhouse-cpp link both 404, and the list of other vendors' clients does not belong here. Four things deliberately keep the vendor name, and CI records why: LICENSE and AUTHORS are the copyright notices Apache-2.0 requires; db.system in query.go is an OpenTelemetry enum whose registered value for this DBMS is fixed by the specification, so emitting anything else would be non-conformant; and README names clickhouse-local, which is genuinely the binary our own server image ships. Numeric error codes and protocol revision constants are untouched.
126 lines
2.5 KiB
YAML
126 lines
2.5 KiB
YAML
version: "2"
|
|
linters:
|
|
default: none
|
|
enable:
|
|
- dogsled
|
|
- errcheck
|
|
- goconst
|
|
- gocritic
|
|
- gosec
|
|
- govet
|
|
- ineffassign
|
|
- misspell
|
|
- nakedret
|
|
- revive
|
|
- staticcheck
|
|
- unconvert
|
|
- unparam
|
|
- whitespace
|
|
settings:
|
|
dupl:
|
|
threshold: 120
|
|
goconst:
|
|
min-len: 2
|
|
min-occurrences: 3
|
|
gocritic:
|
|
disabled-checks:
|
|
- hugeParam
|
|
- rangeValCopy
|
|
- exitAfterDefer
|
|
- whyNoLint
|
|
- singleCaseSwitch
|
|
- commentedOutCode
|
|
- appendAssign
|
|
- unnecessaryBlock
|
|
- redundantSprint
|
|
- octalLiteral
|
|
- ifElseChain
|
|
enabled-tags:
|
|
- diagnostic
|
|
- experimental
|
|
- opinionated
|
|
- performance
|
|
- style
|
|
gocyclo:
|
|
min-complexity: 15
|
|
gosec:
|
|
excludes:
|
|
- G115
|
|
govet:
|
|
enable:
|
|
- shadow
|
|
lll:
|
|
line-length: 140
|
|
misspell:
|
|
locale: US
|
|
revive:
|
|
rules:
|
|
- name: unused-parameter
|
|
disabled: true
|
|
exclusions:
|
|
generated: strict
|
|
rules:
|
|
- linters:
|
|
- revive
|
|
text: should have comment
|
|
- linters:
|
|
- dupl
|
|
- errcheck
|
|
- funlen
|
|
- gocognit
|
|
- goconst
|
|
- gocyclo
|
|
- gosec
|
|
- lll
|
|
- scopelint
|
|
path: _test\.go
|
|
- linters:
|
|
- gosec
|
|
path: internal/cht
|
|
- linters:
|
|
- gocritic
|
|
path: _test\.go
|
|
text: Combine
|
|
- linters:
|
|
- gocritic
|
|
path: _test\.go
|
|
text: dupArg
|
|
source: (assert|require).+Equal
|
|
- linters:
|
|
- govet
|
|
text: declaration of "(err|ctx|log|lg|c)"
|
|
- linters:
|
|
- revive
|
|
text: 'receiver-naming: receiver name \S+ should be consistent with previous receiver name \S+ for invalid-type'
|
|
- linters:
|
|
- funlen
|
|
- gocognit
|
|
- goconst
|
|
- gocyclo
|
|
path: main\.go
|
|
- linters:
|
|
- gosec
|
|
path: main\.go
|
|
text: G307
|
|
- linters:
|
|
- staticcheck
|
|
text: ST1003
|
|
source: func Fuzz.+\(f \*testing\.F\)
|
|
- linters:
|
|
- revive
|
|
path: (internal|cmd)
|
|
text: package-comments
|
|
- linters:
|
|
- revive
|
|
text: comment on exported const .+ should be of the form
|
|
formatters:
|
|
enable:
|
|
- gofmt
|
|
- goimports
|
|
settings:
|
|
goimports:
|
|
local-prefixes:
|
|
- github.com/hanzo-ds/native
|
|
exclusions:
|
|
generated: strict
|