fix: Add nil check for cronjobRef.Controller (#95) #30
Annotations
10 errors and 11 warnings
|
informer_jobs.go#L39
Error return value is not checked (errcheck)
|
|
watcher_events.go#L254
Error return value is not checked (errcheck)
|
|
watcher_pods.go#L196
Error return value is not checked (errcheck)
|
|
sentry_dsn_data_test.go#L35
string `https://c6f9a148ee0775891414b50b9af35959@o4506191942320128.ingest.sentry.io/1234567890` has 4 occurrences, make it a constant (goconst)
|
|
crons_monitor_data_test.go#L39
string `* * * * *` has 5 occurrences, make it a constant (goconst)
|
|
crons_monitor_data_test.go#L98
string `TestAddCronsMonitorDataCronJob` has 3 occurrences, make it a constant (goconst)
|
|
crons_monitor_data_test.go#L12
string `080181f33ca343f89b0bf55d50abfeee` has 3 occurrences, make it a constant (goconst)
|
|
crons.go#L56
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
crons.go#L123
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
informer_cronjobs.go#L14
createCronjobInformer - result 1 (error) is always nil (unparam)
|
|
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@3a919529898de77ec3da873e3063ca4b10e7f5cc. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
crons.go#L122
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (revive)
|
|
mocks_test.go#L15
unused-parameter: parameter 'options' seems to be unused, consider removing or renaming it as _ (revive)
|
|
crons.go#L23
unused-parameter: parameter 'eventHandlerType' seems to be unused, consider removing or renaming it as _ (revive)
|
|
mocks_test.go#L21
unused-parameter: parameter 'timeout' seems to be unused, consider removing or renaming it as _ (revive)
|
|
sentry.go#L12
unused-parameter: parameter 'hint' seems to be unused, consider removing or renaming it as _ (revive)
|
|
enhancers.go#L196
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
enhancers.go#L225
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
enhancers.go#L259
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
enhancers_common_test.go#L7
unused-parameter: parameter 't' seems to be unused, consider removing or renaming it as _ (revive)
|
|
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)
|
This job failed
Loading