Replace gometalinter with Golangci lint by silvin-lubecki · Pull Request #1797 · docker/cli

opts/throttledevice.go:51:5: SA4003: unsigned values are never < 0 (staticcheck)

silvin-lubecki

committed

cli/command/trust/inspect_pretty_test.go:399:24: SA4010: this result of append is never used, except maybe in other appends (staticcheck)

silvin-lubecki

committed

cli/command/container/stats.go:211:21: SA1015: using time.Tick leaks the underlying ticker, consider using it only in endless functions, tests and the main package, and use time.NewTicker here (sta…

silvin-lubecki

committed

SA1019: httputil.ErrPersistEOF is deprecated: No longer used. (staticcheck)

silvin-lubecki

committed

cli/command/trust/key_generate.go:112:9: nilness: impossible condition: nil != nil (govet)

silvin-lubecki

committed

cli/command/stack/kubernetes/list.go:32:47: nilness: tautological condition: non-nil != nil (govet)

silvin-lubecki

committed

cli/command/container/start.go:157:20: nilness: nil dereference in type assertion (govet)

silvin-lubecki

committed

cli/registry/client/fetcher.go:106:9: nilness: impossible condition: nil != nil (govet)

silvin-lubecki

committed

cli/compose/types/types.go:106:2: structtag: struct field tag `yaml:",inline", json:"-"` not compatible with reflect.StructTag.Get: key:"value" pairs not separated by spaces (govet)

silvin-lubecki

committed

opts/ulimit_test.go:11:13: composites: `*github.com/docker/cli/vendor/github.com/docker/go-units.Ulimit` composite literal uses unkeyed fields (govet)

silvin-lubecki

committed

cli/command/container/attach.go:141:15: nilness: impossible condition: nil != nil (govet)

silvin-lubecki

committed

unchecked errors

silvin-lubecki

committed

cli/command/image/build/context_test.go:244:38: `createTestTempDir` - `dir` always receives `""` (unparam)

silvin-lubecki

committed

cli/command/image/build/context_test.go:252:71: `createTestTempFile` - `perm` always receives `0777` (`511`) (unparam)

silvin-lubecki

committed

cli/command/image/build_buildkit.go:450:56: parseSSH - result 1 (error) is always nil (unparam)

silvin-lubecki

committed

cli/command/image/build_session.go:133:45: getBuildSharedKey - result 1 (error) is always nil (unparam)

silvin-lubecki

committed

cli/command/plugin/list_test.go:61:31: `TestList$1` - `filter` is unused (unparam)

silvin-lubecki

committed

cli/command/stack/kubernetes/deploy_test.go:65:68: `checkOwnerReferences` - `stackName` always receives `"test"` (unparam)

silvin-lubecki

committed

cli/command/system/info.go:116:68: prettyPrintClientInfo - result 0 (error) is always nil (unparam)

silvin-lubecki

committed

cli/compose/convert/service.go:592:76: convertDNSConfig - result 1 (error) is always nil (unparam)

silvin-lubecki

committed

disable unparam linter on these functions, as we need an error in these function signatures

silvin-lubecki

committed

cli/context/kubernetes/endpoint_test.go:122:38: `checkClientConfig` - `s` is unused (unparam)

silvin-lubecki

committed

Disable unparam linter: cli/required.go:102:16: `pluralize` - `word` always receives `"argument"` (unparam)

silvin-lubecki

committed

Disable unparam linter: e2e/image/push_test.go:299:27: `withNotaryPassphrase` - `pwd` always receives `"foo"` (unparam)

silvin-lubecki

committed

cli/command/trust/sign_test.go:119:70: unnecessary conversion (unconvert)

silvin-lubecki

committed

File is not `goimports`-ed (goimports)

silvin-lubecki

committed

cli/compose/convert/service_test.go:274:72: unnecessary conversion (unconvert)

silvin-lubecki

committed

cli/command/registry/login_test.go:66:25: unnecessary conversion (unconvert)

silvin-lubecki

committed

cli/command/image/build/context_test.go:244:38: `createTestTempDir` - `prefix` always receives `"builder-context-test"` (unparam)

silvin-lubecki

committed

Remove now obsolete gometalinter and use golangci-lint instead with almost the same configuration.

silvin-lubecki

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Replace gometalinter with Golangci lint#1797

Replace gometalinter with Golangci lint#1797
silvin-lubecki wants to merge 30 commits intodocker:masterfrom
silvin-lubecki:golangci-lint

Commits on Apr 2, 2019

opts/throttledevice.go:51:5: SA4003: unsigned values are never < 0 (staticcheck)

cli/command/trust/inspect_pretty_test.go:399:24: SA4010: this result of append is never used, except maybe in other appends (staticcheck)

cli/command/container/stats.go:211:21: SA1015: using time.Tick leaks the underlying ticker, consider using it only in endless functions, tests and the main package, and use time.NewTicker here (sta…

SA1019: httputil.ErrPersistEOF is deprecated: No longer used. (staticcheck)

cli/command/trust/key_generate.go:112:9: nilness: impossible condition: nil != nil (govet)

cli/command/stack/kubernetes/list.go:32:47: nilness: tautological condition: non-nil != nil (govet)

cli/command/container/start.go:157:20: nilness: nil dereference in type assertion (govet)

cli/registry/client/fetcher.go:106:9: nilness: impossible condition: nil != nil (govet)

cli/compose/types/types.go:106:2: structtag: struct field tag `yaml:",inline", json:"-"` not compatible with reflect.StructTag.Get: key:"value" pairs not separated by spaces (govet)

opts/ulimit_test.go:11:13: composites: `*github.com/docker/cli/vendor/github.com/docker/go-units.Ulimit` composite literal uses unkeyed fields (govet)

cli/command/container/attach.go:141:15: nilness: impossible condition: nil != nil (govet)

unchecked errors

cli/command/image/build/context_test.go:244:38: `createTestTempDir` - `dir` always receives `""` (unparam)

cli/command/image/build/context_test.go:252:71: `createTestTempFile` - `perm` always receives `0777` (`511`) (unparam)

cli/command/image/build_buildkit.go:450:56: parseSSH - result 1 (error) is always nil (unparam)

cli/command/image/build_session.go:133:45: getBuildSharedKey - result 1 (error) is always nil (unparam)

cli/command/plugin/list_test.go:61:31: `TestList$1` - `filter` is unused (unparam)

cli/command/stack/kubernetes/deploy_test.go:65:68: `checkOwnerReferences` - `stackName` always receives `"test"` (unparam)

cli/command/system/info.go:116:68: prettyPrintClientInfo - result 0 (error) is always nil (unparam)

cli/compose/convert/service.go:592:76: convertDNSConfig - result 1 (error) is always nil (unparam)

disable unparam linter on these functions, as we need an error in these function signatures

cli/context/kubernetes/endpoint_test.go:122:38: `checkClientConfig` - `s` is unused (unparam)

Disable unparam linter: cli/required.go:102:16: `pluralize` - `word` always receives `"argument"` (unparam)

Disable unparam linter: e2e/image/push_test.go:299:27: `withNotaryPassphrase` - `pwd` always receives `"foo"` (unparam)

cli/command/trust/sign_test.go:119:70: unnecessary conversion (unconvert)

File is not `goimports`-ed (goimports)

cli/compose/convert/service_test.go:274:72: unnecessary conversion (unconvert)

cli/command/registry/login_test.go:66:25: unnecessary conversion (unconvert)

cli/command/image/build/context_test.go:244:38: `createTestTempDir` - `prefix` always receives `"builder-context-test"` (unparam)

Remove now obsolete gometalinter and use golangci-lint instead with almost the same configuration.