Skip to content

fix: Remove the Datadog logging adapter #229

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nhhagen
Copy link
Member

@nhhagen nhhagen commented Jun 23, 2025

This change removes the Datadog logging adapter. The adapter is only
consumed by go-datalog-lib and should not have been implemented in
go-logger.

With the V2 changes to dd-trace-go removing the sooner than later will
make migration to OpenTelemetry or dd-trace-go V2 simpler.

The functionality is copied into go-datadog-lib, see
coopnorge/go-datadog-lib#571 for more
information.

This change removes the Datadog logging adapter. The adapter is only
consumed by `go-datalog-lib` and should not have been implemented in
`go-logger`.

With the V2 changes to `dd-trace-go` removing the sooner than later will
make migration to OpenTelemetry or `dd-trace-go` V2 simpler.

The functionality is copied into `go-datadog-lib`, see
[coopnorge/go-datadog-lib#571][coopnorge/go-datadog-lib#571] for more
information.

[coopnorge/go-datadog-lib#571]: coopnorge/go-datadog-lib#571
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant