Skip to content

Commit

Permalink
add lint
Browse files Browse the repository at this point in the history
  • Loading branch information
golangisfun123 committed Sep 4, 2024
1 parent 9632578 commit c762c4c
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions core/metrics/otlp.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,20 @@ func NewOTLPMetricsHandler(buildInfo config.BuildInfo) Handler {

func (n *otlpHandler) Start(ctx context.Context) (err error) {
// TODO: generalize this to allow for more than two exporters.
client, err := buildClientFromTransport(transportFromString(core.GetEnv(otlpTransportEnv, otlpTransportGRPC.String())))
client, err := buildClientFromTransport(
transportFromString(
core.GetEnv(otlpTransportEnv, otlpTransportGRPC.String()),
),
)
if err != nil {
return fmt.Errorf("could not create client: %w", err)
}

Check warning on line 39 in core/metrics/otlp.go

View check run for this annotation

Codecov / codecov/patch

core/metrics/otlp.go#L38-L39

Added lines #L38 - L39 were not covered by tests

secondaryClient, err := buildClientFromTransport(transportFromString(core.GetEnv(otlpTransportEnvSecondary, otlpTransportHTTP.String())))
secondaryClient, err := buildClientFromTransport(
transportFromString(
core.GetEnv(otlpTransportEnvSecondary, otlpTransportGRPC.String()),
),
)
if err != nil {
return fmt.Errorf("could not create secondary client: %w", err)

Check warning on line 47 in core/metrics/otlp.go

View check run for this annotation

Codecov / codecov/patch

core/metrics/otlp.go#L47

Added line #L47 was not covered by tests
}
Expand Down

0 comments on commit c762c4c

Please sign in to comment.