chore(deps): upgrade {logger, go version, linters} #3
narcis96Annotations
10 errors and 11 warnings
|
lint:
pkg/helm/repository/repository.go#L85
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
lint:
controllers/vcluster_controller.go#L262
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
lint:
pkg/helm/secrets.go#L147
Consider pre-allocating `results` (prealloc)
|
|
lint:
pkg/helm/labels.go#L7
func `labels.get` is unused (unused)
|
|
lint:
pkg/helm/labels.go#L8
func `labels.set` is unused (unused)
|
|
lint:
pkg/helm/labels.go#L10
func `labels.keys` is unused (unused)
|
|
lint:
pkg/helm/labels.go#L17
func `labels.match` is unused (unused)
|
|
lint:
pkg/helm/labels.go#L26
func `labels.toMap` is unused (unused)
|
|
lint:
pkg/helm/labels.go#L28
func `(*labels).fromMap` is unused (unused)
|
|
lint:
pkg/helm/secrets.go#L219
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
|
|
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
lint:
pkg/compress/compress_test.go#L5
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
pkg/util/cidrdiscovery/cidr.go#L16
unexported-return: exported func NewCIDRLookup returns unexported type *cidrdiscovery.serviceCIDR, which can be annoying to use (revive)
|
|
lint:
pkg/util/cidrdiscovery/cidr.go#L57
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
pkg/util/cidrdiscovery/cidr.go#L52
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
pkg/helm/types.go#L3
exported: type name will be used as helm.HelmChart by other packages, and that stutters; consider calling this Chart (revive)
|
|
lint:
pkg/helm/types.go#L17
exported: type name will be used as helm.HelmChartRepository by other packages, and that stutters; consider calling this ChartRepository (revive)
|
|
lint:
pkg/helm/helm.go#L38
var-naming: struct field InsecureSkipTlsVerify should be InsecureSkipTLSVerify (revive)
|
|
lint:
pkg/helm/repository/repository.go#L23
exported: type name will be used as repository.RepositoryEntries by other packages, and that stutters; consider calling this Entries (revive)
|
|
lint:
pkg/helm/repository/repository.go#L31
exported: type name will be used as repository.RepositoryDefinition by other packages, and that stutters; consider calling this Definition (revive)
|
|
lint:
controllers/vcluster_controller.go#L213
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|