add njs to nginx build · kubernetes/ingress-nginx@4249cef

Triggered via pull request April 10, 2024 19:17

strongjz

synchronize #11248

strongjz:add-njs

Status Failure

Total duration 5m 19s

Artifacts –

golangci-lint.yml

on: pull_request

5 errors

lint: internal/ingress/controller/config/config.go#L455

commentFormatting: put a space between `//` and comment text (gocritic)

lint: pkg/flags/flags.go#L231

File is not `gofmt`-ed with `-s` (gofmt)

lint: internal/ingress/controller/config/config.go#L456

structtag: struct field tag `json:"enable-njs,omitempty` not compatible with reflect.StructTag.Get: bad syntax for struct tag value (govet)

lint: internal/ingress/controller/template/template.go#L193

the given struct should be annotated with the `json` tag (musttag)

issues found