Skip to content

Merge pull request #31 from TheRealSibasishBehera/main #72

Merge pull request #31 from TheRealSibasishBehera/main

Merge pull request #31 from TheRealSibasishBehera/main #72

Triggered via push July 10, 2023 05:39
Status Success
Total duration 2m 18s
Artifacts

ci-go.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
go-lint: relay-server/main.go#L57
var esUrl should be esURL
go-lint: relay-server/common/common.go#L4
should have a package comment
go-lint: relay-server/main.go#L4
should have a package comment
go-lint: relay-server/log/logger.go#L4
should have a package comment
go-lint: relay-server/elasticsearch/adapter.go#L1
should have a package comment
go-lint: relay-server/server/k8sHandler.go#L4
should have a package comment
go-lint: relay-server/elasticsearch/adapter.go#L29
type name will be used as elasticsearch.ElasticsearchClient by other packages, and that stutters; consider calling this Client
go-lint: relay-server/server/relayServer.go#L511
redundant if ...; err != nil check, just return error instead.
go-lint: relay-server/server/relayServer.go#L92
parameter 'ctx' seems to be unused, consider removing or renaming it as _
go-lint: relay-server/server/relayServer.go#L624
exported function DeleteClientEntry should have comment or be unexported