ci: add pr checks workflow #9
Merged
Annotations
10 errors
|
internal/state/state.go#L9
blank-imports: a blank import should be only in a main or test package, or have a comment justifying it (revive)
|
|
internal/slurm/slurm.go#L40
exported: type name will be used as slurm.SlurmJob by other packages, and that stutters; consider calling this Job (revive)
|
|
internal/config/config.go#L24
exported: type name will be used as config.ConfigFile by other packages, and that stutters; consider calling this File (revive)
|
|
cmd/slurm-tracker/main.go#L111
shadow: declaration of "err" shadows declaration at line 106 (govet)
|
|
internal/tracker/tracker.go#L36
redundantSprint: job.State.Current is already string (gocritic)
|
|
internal/tracker/tracker.go#L28
hugeParam: cfg is heavy (128 bytes); consider passing it by pointer (gocritic)
|
|
internal/slurm/slurm.go#L178
hugeParam: job is heavy (544 bytes); consider passing it by pointer (gocritic)
|
|
internal/slurm/slurm.go#L164
hugeParam: job is heavy (544 bytes); consider passing it by pointer (gocritic)
|
|
internal/slurm/slurm.go#L159
hugeParam: job is heavy (544 bytes); consider passing it by pointer (gocritic)
|
|
cmd/slurm-tracker/main.go#L127
rangeValCopy: each iteration copies 544 bytes (consider pointers or indexing) (gocritic)
|
The logs for this run have expired and are no longer available.
Loading