ci: Add golangi-lint configuration (#77) #7
tonyoAnnotations
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:
crons_monitor_data_test.go#L39
string `* * * * *` has 5 occurrences, make it a constant (goconst)
|
|
lint:
crons_monitor_data_test.go#L95
string `TestAddCronsMonitorDataCronJob` has 3 occurrences, make it a constant (goconst)
|
|
lint:
enhancers_common_test.go#L7
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
utils.go#L24
var-naming: func prettyJson should be prettyJSON (revive)
|
|
lint:
utils.go#L12
var-declaration: should omit type map[string]struct{} from declaration of var truthyStrings; it will be inferred from the right-hand side (revive)
|
|
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:
informer_jobs.go#L14
unused-parameter: parameter 'namespace' seems to be unused, consider removing or renaming it as _ (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:
crons_monitor_data.go#L12
var-naming: struct field CheckinId should be CheckinID (revive)
|
|
lint:
crons_monitor_data.go#L16
var-naming: func parameter checkinId should be checkinID (revive)
|
|
lint:
crons_monitor_data.go#L22
var-naming: method getCheckinId should be getCheckinID (revive)
|
|
lint:
crons_monitor_data.go#L58
var-naming: method parameter checkinId should be checkinID (revive)
|