✨ WIP: DO NOT MERGE: Shift Week Prototypes · operator-framework/operator-controller@d0801a5

Triggered via pull request November 8, 2024 20:43

joelanford

synchronize #1418

joelanford:shift-week

Status Failure

Total duration 3m 20s

Artifacts –

sanity.yaml

on: pull_request

10 errors

lint: internal/rukpak/convert/v2/registryv1.go#L198

Consider pre-allocating `chrtMaintainers` (prealloc)

lint: internal/rukpak/convert/v2/registryv1.go#L1185

`getWatchNamespacesSchema$2` - `maxLength` always receives `10` (unparam)

lint: internal/rukpak/convert/v2/registryv1_helpers_test.go#L97

assertFieldEqual - result 0 (bool) is never used (unparam)

lint: internal/rukpak/convert/v2/registryv1_test.go#L57

`TestRegistryV1ToHelmChart$1` - `chrt` is unused (unparam)

lint: internal/rukpak/convert/v2/registryv1_test.go#L117

`TestRegistryV1ToHelmChart$2` - `chrt` is unused (unparam)

lint: internal/rukpak/convert/v2/registryv1_test.go#L54

field `annotations` is unused (unused)

lint: internal/rukpak/convert/v2/registryv1_test.go#L773

unnecessary trailing newline (whitespace)

lint: internal/rukpak/convert/v2/registryv1_test.go#L587

require-error: for error assertions use require (testifylint)

lint: internal/rukpak/convert/v2/registryv1_test.go#L589

require-error: for error assertions use require (testifylint)

lint: internal/rukpak/convert/v2/registryv1_test.go#L795

require-error: for error assertions use require (testifylint)