Skip to content

fix: Enable more linters #8

fix: Enable more linters

fix: Enable more linters #8

Triggered via pull request March 5, 2024 14:52
Status Failure
Total duration 3m 13s
Artifacts

ci.yml

on: pull_request
container-check
1m 10s
container-check
project-check
27s
project-check
Matrix: golang-check
Fit to window
Zoom out
Zoom in

Annotations

32 errors
golang-check (1.22): cmd/main.go#L17
package-comments: should have a package comment (revive)
golang-check (1.22): cmd/main.go#L55
function-length: maximum number of lines per function exceeded; max 75 but got 101 (revive)
golang-check (1.22): internal/controller/common_k8s.go#L17
package-comments: should have a package comment (revive)
golang-check (1.22): internal/controller/common_k8s.go#L37
exported: exported const OadpNamespace should have comment (or a comment on this block) or be unexported (revive)
golang-check (1.22): internal/controller/common_k8s.go#L40
exported: exported function CreateLabelsForNac should have comment or be unexported (revive)
golang-check (1.22): internal/controller/common_k8s.go#L55
exported: exported function CreateAnnotationsForNac should have comment or be unexported (revive)
golang-check (1.22): internal/controller/common_k8s.go#L72
cognitive-complexity: function mergeUniqueKeyTOfTMaps has cognitive complexity 14 (> max enabled 7) (revive)
golang-check (1.22): internal/controller/common_k8s_test.go#L26
cognitive-complexity: function TestMergeUniqueKeyTOfTMaps has cognitive complexity 8 (> max enabled 7) (revive)
golang-check (1.22): internal/controller/common_nab.go#L17
ST1000: at least one file in a package should have a package comment (stylecheck)
golang-check (1.22): internal/controller/common_nab.go#L20
G505: Blocklisted import crypto/sha1: weak cryptographic primitive (gosec)
golang-check (1.20)
The job was canceled because "_1_22" failed.
golang-check (1.20): cmd/main.go#L17
package-comments: should have a package comment (revive)
golang-check (1.20): cmd/main.go#L55
function-length: maximum number of lines per function exceeded; max 75 but got 101 (revive)
golang-check (1.20): internal/controller/common_k8s.go#L17
package-comments: should have a package comment (revive)
golang-check (1.20): internal/controller/common_k8s.go#L37
exported: exported const OadpNamespace should have comment (or a comment on this block) or be unexported (revive)
golang-check (1.20): internal/controller/common_k8s.go#L40
exported: exported function CreateLabelsForNac should have comment or be unexported (revive)
golang-check (1.20): internal/controller/common_k8s.go#L55
exported: exported function CreateAnnotationsForNac should have comment or be unexported (revive)
golang-check (1.20): internal/controller/common_k8s.go#L72
cognitive-complexity: function mergeUniqueKeyTOfTMaps has cognitive complexity 14 (> max enabled 7) (revive)
golang-check (1.20): internal/controller/common_k8s_test.go#L26
cognitive-complexity: function TestMergeUniqueKeyTOfTMaps has cognitive complexity 8 (> max enabled 7) (revive)
golang-check (1.20): internal/controller/common_nab.go#L17
ST1000: at least one file in a package should have a package comment (stylecheck)
golang-check (1.20): internal/controller/common_nab.go#L20
G505: Blocklisted import crypto/sha1: weak cryptographic primitive (gosec)
golang-check (1.21)
The job was canceled because "_1_22" failed.
golang-check (1.21): cmd/main.go#L17
package-comments: should have a package comment (revive)
golang-check (1.21): cmd/main.go#L55
function-length: maximum number of lines per function exceeded; max 75 but got 101 (revive)
golang-check (1.21): internal/controller/common_k8s.go#L17
package-comments: should have a package comment (revive)
golang-check (1.21): internal/controller/common_k8s.go#L37
exported: exported const OadpNamespace should have comment (or a comment on this block) or be unexported (revive)
golang-check (1.21): internal/controller/common_k8s.go#L40
exported: exported function CreateLabelsForNac should have comment or be unexported (revive)
golang-check (1.21): internal/controller/common_k8s.go#L55
exported: exported function CreateAnnotationsForNac should have comment or be unexported (revive)
golang-check (1.21): internal/controller/common_k8s.go#L72
cognitive-complexity: function mergeUniqueKeyTOfTMaps has cognitive complexity 14 (> max enabled 7) (revive)
golang-check (1.21): internal/controller/common_k8s_test.go#L26
cognitive-complexity: function TestMergeUniqueKeyTOfTMaps has cognitive complexity 8 (> max enabled 7) (revive)
golang-check (1.21): internal/controller/common_nab.go#L17
ST1000: at least one file in a package should have a package comment (stylecheck)
golang-check (1.21): internal/controller/common_nab.go#L20
G505: Blocklisted import crypto/sha1: weak cryptographic primitive (gosec)