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

go.mod, go.sum: upgrade to remove old dependencies #3836

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

Conversation

jba
Copy link

@jba jba commented Aug 12, 2024

Upgrade cobra, prometheus and golang.org/x/perf to the latest versions. This removes a lot of clutter from go.sum, including the spiral of old cloud.google.com/go versions. Even though these old versions aren't part of the build, the go command is forced to fetch their go.mod files.

Upgrade cobra, prometheus and golang.org/x/perf to the latest versions.
This removes a lot of clutter from go.sum, including the spiral of old
cloud.google.com/go versions. Even though these old versions aren't part
of the build, the go command is forced to fetch their go.mod files.
@jba jba requested a review from a team as a code owner August 12, 2024 13:27
@jba jba requested a review from jbowens August 12, 2024 13:27
@cockroach-teamcity
Copy link
Member

This change is Reviewable

github.com/prometheus/common v0.32.1 // indirect
github.com/prometheus/procfs v0.7.3 // indirect
github.com/rogpeppe/go-internal v1.9.0 // indirect
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this dep coming from, and what is it used for?

Copy link
Author

Choose a reason for hiding this comment

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

go mod why says:

github.com/cockroachdb/pebble
github.com/prometheus/client_golang/prometheus                                                                                               
github.com/prometheus/common/expfmt
github.com/munnerz/goautoneg  

The expfmt package uses it twice:

ParseAccept parses an accept header:
https://github.com/munnerz/goautoneg/blob/master/autoneg.go#L93

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.

3 participants