Skip to content

Merge pull request #401 from Jarily/master #821

Merge pull request #401 from Jarily/master

Merge pull request #401 from Jarily/master #821

Triggered via push June 24, 2024 02:28
Status Failure
Total duration 1m 16s
Artifacts
Fit to window
Zoom out
Zoom in

Annotations

6 errors and 1 warning
lint: server/egovernor/container.go#L59
Error return value of `obj.Update` is not checked (errcheck)
lint: test/gracefulstop/main.go#L19
S1023: redundant `return` statement (gosimple)
lint: client/egrpc/interceptor.go#L393
SA1029: should not use built-in type string as key for value; define your own type to avoid collisions (staticcheck)
lint: core/transport/transport.go#L46
SA1029: should not use built-in type string as key for value; define your own type to avoid collisions (staticcheck)
lint: core/etrace/otel/config.go#L6
SA1019: "go.opentelemetry.io/otel/exporters/jaeger" is deprecated: This module is no longer supported. OpenTelemetry dropped support for Jaeger exporter in July 2023. Jaeger officially accepts and recommends using OTLP. Use [go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp] or [go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc] instead. (staticcheck)
lint
issues found
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v4, golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.