Skip to content

Add linter

Add linter #19

Triggered via pull request June 2, 2023 09:55
Status Failure
Total duration 1m 34s
Artifacts

linter.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 11 warnings
linter: pkg/kubehound/storage/cache/memcache_bench_test.go#L36
Error return value of `fakeCacheWriter.Queue` is not checked (errcheck)
linter: pkg/kubehound/storage/cache/memcache_bench_test.go#L66
Error return value of `fakeProvider.Get` is not checked (errcheck)
linter: pkg/kubehound/storage/cache/memcache_test.go#L21
Error return value of `fakeCacheWriter.Queue` is not checked (errcheck)
linter: pkg/kubehound/ingestor/pipeline/ingest_resources.go#L140
Error return value of `i.cleanupAll` is not checked (errcheck)
linter: pkg/kubehound/storage/cache/memcache_bench_test.go#L65
File is not `gofmt`-ed with `-s` (gofmt)
linter: pkg/kubehound/store/collections/container.go#L3
File is not `gofumpt`-ed (gofumpt)
linter: pkg/kubehound/store/collections/identity.go#L3
File is not `gofumpt`-ed (gofumpt)
linter: pkg/kubehound/store/collections/node.go#L3
File is not `gofumpt`-ed (gofumpt)
linter: pkg/collector/file.go#L97
G601: Implicit memory aliasing in for loop. (gosec)
linter: pkg/collector/file.go#L135
G601: Implicit memory aliasing in for loop. (gosec)
linter: pkg/kubehound/models/graph/models.go#L12
var-naming: struct field StoreId should be StoreID (revive)
linter: pkg/kubehound/models/graph/models.go#L33
var-naming: struct field StoreId should be StoreID (revive)
linter: pkg/kubehound/models/graph/models.go#L38
var-naming: struct field StoreId should be StoreID (revive)
linter: pkg/kubehound/graph/vertex/vertex.go#L11
exported: type name will be used as vertex.VertexTraversal by other packages, and that stutters; consider calling this Traversal (revive)
linter: pkg/telemetry/log/logger.go#L17
var-declaration: should omit type *KubehoundLogger from declaration of var I; it will be inferred from the right-hand side (revive)
linter: pkg/kubehound/storage/cache/memcache.go#L32
receiver-naming: receiver name m should be consistent with previous receiver name mp for MemCacheProvider (revive)
linter: pkg/kubehound/storage/cache/memcache.go#L37
receiver-naming: receiver name m should be consistent with previous receiver name mp for MemCacheProvider (revive)
linter: pkg/kubehound/storage/cache/memcache.go#L41
receiver-naming: receiver name m should be consistent with previous receiver name mp for MemCacheProvider (revive)
linter: pkg/kubehound/storage/cache/container_key.go#L14
unexported-return: exported func ContainerKey returns unexported type *cache.containerCacheKey, which can be annoying to use (revive)
linter: pkg/kubehound/storage/cache/keys.go#L4
exported: type name will be used as cache.CacheKey by other packages, and that stutters; consider calling this Key (revive)
linter
Restore cache failed: Dependencies file is not found in /home/runner/work/KubeHound/KubeHound. Supported file pattern: go.sum