test: update TestRunEnhancers (#82) #17
Jiahui-Zhang-20Annotations
10 errors and 10 warnings
|
lint:
crons.go#L58
Error return value is not checked (errcheck)
|
|
lint:
crons.go#L62
Error return value is not checked (errcheck)
|
|
lint:
crons.go#L94
Error return value of `cronsMonitorData.addJob` is not checked (errcheck)
|
|
lint:
crons_monitor_data_test.go#L71
Error return value of `cronsMonitorData.addJob` is not checked (errcheck)
|
|
lint:
enhancers_common.go#L125
Error return value is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L42
Error return value of `clientMapping.AddClientToMap` is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L52
Error return value of `clientMapping.AddClientToMap` is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L70
Error return value of `clientMapping.AddClientToMap` is not checked (errcheck)
|
|
lint:
sentry_dsn_data_test.go#L35
string `https://c6f9a148ee0775891414b50b9af35959@o4506191942320128.ingest.sentry.io/1234567890` has 4 occurrences, make it a constant (goconst)
|
|
lint:
crons_monitor_data_test.go#L12
string `080181f33ca343f89b0bf55d50abfeee` has 3 occurrences, make it a constant (goconst)
|
|
lint:
context.go#L28
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
|
lint:
k8s.go#L71
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
cluster_config.go#L53
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
cluster_config.go#L83
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
lint:
version.go#L3
var-declaration: should omit type string from declaration of var version; it will be inferred from the right-hand side (revive)
|
|
lint:
enhancers.go#L93
var-declaration: should drop = nil from declaration of var err; it is the zero value (revive)
|
|
lint:
informer_deployments.go#L11
unused-parameter: parameter 'namespace' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
enhancers_common.go#L65
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
enhancers_common.go#L92
unused-parameter: parameter 'scope' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
enhancers.go#L59
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|