Skip to content

Cleanup on Preemption #326

Cleanup on Preemption

Cleanup on Preemption #326

Triggered via pull request August 15, 2023 17:38
Status Failure
Total duration 1m 45s
Artifacts 1

build.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

9 errors and 5 warnings
golangci-lint: cmd/arcaflow/main.go#L196
commentFormatting: put a space between `//` and comment text (gocritic)
golangci-lint: cmd/arcaflow/main.go#L222
commentFormatting: put a space between `//` and comment text (gocritic)
golangci-lint: cmd/arcaflow/main.go#L225
commentFormatting: put a space between `//` and comment text (gocritic)
golangci-lint: cmd/arcaflow/main.go#L49
Comment should end in a period (godot)
golangci-lint: cmd/arcaflow/main.go#L416
func `osSignalToInt` is unused (unused)
golangci-lint: cmd/arcaflow/main.go#L254
SA1016: os.Kill cannot be trapped (did you mean syscall.SIGTERM?) (staticcheck)
golangci-lint: workflow/workflow.go#L50
directive `//nolint:gocognit` is unused for linter "gocognit" (nolintlint)
golangci-lint
issues found
go test
Process completed with exit code 1.
golangci-lint: engine.go#L39
exported: exported type WorkflowResult should have comment or be unexported (revive)
golangci-lint: cmd/arcaflow/main.go#L396
unused-parameter: parameter 'cancel' seems to be unused, consider removing or renaming it as _ (revive)
golangci-lint: cmd/arcaflow/main.go#L59
exported: exported type ArcaflowExitSignal should have comment or be unexported (revive)
golangci-lint: cmd/arcaflow/main.go#L61
exported: exported method ArcaflowExitSignal.Signal should have comment or be unexported (revive)
golangci-lint: cmd/arcaflow/main.go#L410
exported: exported var OK should have comment or be unexported (revive)

Artifacts

Produced during runtime
Name Size
test-results Expired
168 KB