Skip to content

Feature/auth and tls for brokers #4

Feature/auth and tls for brokers

Feature/auth and tls for brokers #4

Triggered via pull request April 10, 2024 23:29
Status Failure
Total duration 5m 30s
Artifacts

ci.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 errors
Check generated
Process completed with exit code 1.
Lint: pkg/extensions/brokers/natsjetstream/natsjetstream_test.go#L26
commentFormatting: put a space between `//` and comment text (gocritic)
Lint: pkg/ci/brokers.go#L142
commentFormatting: put a space between `//` and comment text (gocritic)
Lint: pkg/extensions/brokers/kafka/kafka.go#L135
Comment should end in a period (godot)
Lint: pkg/ci/brokers.go#L171
exported: comment on exported function BrokerNATSSecure should be of the form "BrokerNATSSecure ..." (revive)
Lint: test/v2/issues/169/suite_test.go#L103
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
Lint: test/v2/issues/169/suite_test.go#L128
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
Lint: pkg/extensions/brokers/nats/nats_test.go#L79
SA5001: should check returned error before deferring nb.Close() (staticcheck)
Lint: pkg/extensions/brokers/nats/nats_test.go#L94
SA5001: should check returned error before deferring nb.Close() (staticcheck)
Lint: pkg/extensions/brokers/natsjetstream/natsjetstream_test.go#L109
SA5001: should check returned error before deferring jc.Close() (staticcheck)
Lint: test/v2/issues/169/suite_test.go#L31
SA5001: should check returned error before deferring natsBrokerTLSBasicAuth.Close() (staticcheck)