Skip to content
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

upgrade statsd: github.com/DataDog/datadog-go to v4.8.3+incompatible from v3.7.2+incompatible #1076

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

jmhodges
Copy link
Contributor

@jmhodges jmhodges commented Dec 7, 2024

This doesn't quite get us to a version of datadog-go that fully
supported Go modules (see the "+incompatible" suffix), but this bump
gets us closer.

Inspecting https://pkg.go.dev/github.com/DataDog/datadog-go tells us
there's a v5 version we should be importing:
https://pkg.go.dev/github.com/DataDog/datadog-go/v5. We can do that
after performing this upgrade and seeing it work well.

This change was made by running:

$ go get github.com/DataDog/datadog-go@latest
$ go mod tidy
$ go mod vendor

…7.2+incompatible

This doesn't quite get us to a version of datadog-go that fully
supported Go modules (see the "+incompatible" suffix), but this bump
gets us closer.

Inspecting <https://pkg.go.dev/github.com/DataDog/datadog-go> tells us
there's a v5 version we should be importing:
<https://pkg.go.dev/github.com/DataDog/datadog-go/v5>. We can do that
after performing this upgrade and seeing it work well.

This change was made by running:

```
$ go get github.com/DataDog/datadog-go@latest
$ go mod tidy
$ go mod vendor
```
@jmhodges jmhodges marked this pull request as ready for review December 9, 2024 22:16
@jmhodges jmhodges requested review from a team as code owners December 9, 2024 22:16
@jmhodges jmhodges requested review from alexcottner and removed request for a team December 9, 2024 22:16
@jmhodges jmhodges changed the title upgrade github.com/DataDog/datadog-go to v4.8.3+incompatible from v3.7.2+incompatible upgrade statds: github.com/DataDog/datadog-go to v4.8.3+incompatible from v3.7.2+incompatible Dec 9, 2024
@jmhodges jmhodges changed the title upgrade statds: github.com/DataDog/datadog-go to v4.8.3+incompatible from v3.7.2+incompatible upgrade statsd: github.com/DataDog/datadog-go to v4.8.3+incompatible from v3.7.2+incompatible Dec 9, 2024
Copy link
Contributor

@say-yawn say-yawn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM but quick questions:

  • Why was this update separate from dependabot updates?
  • Why was this package behind on being updated?

@jmhodges
Copy link
Contributor Author

jmhodges commented Dec 9, 2024

Changes LGTM but quick questions:

* Why was this update separate from dependabot updates?

* Why was this package behind on being updated?

@say-yawn A previous maintainer had explicitly closed and paused the updates for this dependency back in 2021 in #803 . That pause was done because he was not a software engineer and couldn't vet the dependency.

This PR will reopen this line of deps to be automatically updated.

I've been slow-rolling version bumps. You can see all the things we're behind on by running:

go get -u
go mod tidy

@jmhodges jmhodges merged commit 9896e20 into main Dec 9, 2024
14 checks passed
@jmhodges jmhodges deleted the upgrade-datadog-explicitly-gnight branch December 9, 2024 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants