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: Exclude prometheus/client_golang v1.12.1 #1027

Merged
merged 1 commit into from Apr 13, 2022

Conversation

mrueg
Copy link
Contributor

@mrueg mrueg commented Apr 11, 2022

See also #1012
As suggested in #1012 (comment)

Copy link
Member

@kakkoyun kakkoyun left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for taking care of this.

I guess Go 1.15 tests are failing, if we don't have a solution for it, I'm ok to drop the support for 1.15 since 1.18 is out.
I think having support for the last 3 major versions of Go is acceptable.

Better to do that in a separate PR though.

@kakkoyun
Copy link
Member

Hey @mrueg, we plan to release soonish? Would you be able to have a look at this so that we can include it in the release?

@mrueg
Copy link
Contributor Author

mrueg commented Apr 13, 2022

Hey @mrueg, we plan to release soonish? Would you be able to have a look at this so that we can include it in the release?

Dropping go 1.15 support? Definitely can look into it.

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks! One nit only.

@@ -4,6 +4,7 @@ require (
github.com/beorn7/perks v1.0.1
github.com/cespare/xxhash/v2 v2.1.2
github.com/golang/protobuf v1.5.2
github.com/jpillora/backoff v1.0.0 // indirect
Copy link
Member

Choose a reason for hiding this comment

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

This looks off. Do you mind doing go mod tidy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah that got added with go mod tidy

go mod why github.com/jpillora/backoff
# github.com/jpillora/backoff
github.com/prometheus/client_golang/api/prometheus/v1
github.com/prometheus/client_golang/api/prometheus/v1.test
github.com/prometheus/common/config
github.com/mwitkow/go-conntrack
github.com/jpillora/backoff

See also prometheus#1012

As suggested in prometheus#1012 (comment)

Signed-off-by: Manuel Rüger <manuel@rueg.eu>
@mrueg
Copy link
Contributor Author

mrueg commented Apr 13, 2022

Rebased on main, and created a PR on common: prometheus/common#371

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks! 💪🏽

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.

None yet

3 participants