Skip to content

Merge pull request #135 from kobergj/FixNatsjsDurableConsumers #70

Merge pull request #135 from kobergj/FixNatsjsDurableConsumers

Merge pull request #135 from kobergj/FixNatsjsDurableConsumers #70

Triggered via push December 14, 2023 15:13
Status Failure
Total duration 13m 36s
Artifacts

main.yaml

on: push
Fit to window
Zoom out
Zoom in

Annotations

17 errors and 10 warnings
Lint: gobreaker.go#L27
line is 121 characters (lll)
Lint: gobreaker.go#L39
only one cuddle assignment allowed before if statement (wsl)
Lint: gobreaker.go#L56
anonymous switch statements should never be cuddled (wsl)
Lint: gobreaker.go#L57
avoid direct access to proto field merr.Code, use merr.GetCode() instead (protogetter)
Lint: gobreaker.go#L59
avoid direct access to proto field merr.Id, use merr.GetId() instead (protogetter)
Lint: gobreaker.go#L63
avoid direct access to proto field merr.Code, use merr.GetCode() instead (protogetter)
Lint: gobreaker.go#L79
return statements should not be cuddled if block has more than two lines (wsl)
Lint: gobreaker.go#L91
return statements should not be cuddled if block has more than two lines (wsl)
Lint: gobreaker_test.go#L42
type assertion on error will fail on wrapped errors. Use errors.As to check for specific errors (errorlint)
Lint: gobreaker_test.go#L43
avoid direct access to proto field merr.Code, use merr.GetCode() instead (protogetter)
Summary
no required module provides package go.opentelemetry.io/otel/unit; to add it:
Summary
impossible type assertion: ipubsub.MessageAckHandler(m).(*psAckHandler)
Summary
cannot use ackh (variable of type *psAckHandler) as type "cloud.google.com/go/internal/pubsub".AckHandler in argument to ipubsub.NewMessage:
Summary
cannot use "quic-go doesn't build on Go 1.18 yet." (untyped string constant) as int value in variable declaration
Summary
no required module provides package go.opentelemetry.io/otel/unit; to add it:
Summary
Process completed with exit code 1.
Unit Tests
Process completed with exit code 1.
Lint: gobreaker.go#L1
package-comments: should have a package comment (revive)
Lint: gobreaker.go#L12
exported: exported type BreakerMethod should have comment or be unexported (revive)
Lint: gobreaker.go#L15
exported: exported const BreakService should have comment (or a comment on this block) or be unexported (revive)
Lint: configure.go#L1
package-comments: should have a package comment (revive)
Lint: validator.go#L1
package-comments: should have a package comment (revive)
Lint: validator.go#L10
exported: exported type Validator should have comment or be unexported (revive)
Lint: validator.go#L14
exported: exported function NewHandlerWrapper should have comment or be unexported (revive)
Lint: victoriametrics.go#L1
package-comments: should have a package comment (revive)
Lint: victoriametrics.go#L16
exported: comment on exported var DefaultMetricPrefix should be of the form "DefaultMetricPrefix ..." (revive)
Lint: victoriametrics.go#L18
exported: comment on exported var DefaultLabelPrefix should be of the form "DefaultLabelPrefix ..." (revive)