Skip to content

Conversation

SuperQ
Copy link
Member

@SuperQ SuperQ commented Jan 7, 2022

  • Use make -C dir instead of cd dir && make
  • Cleanup uncessary working_directory in config.
  • Update Go modules to latest.
  • Drop support for old Go versions.

Signed-off-by: SuperQ [email protected]

@SuperQ
Copy link
Member Author

SuperQ commented Jan 7, 2022

CC @mrueg

@mrueg
Copy link
Contributor

mrueg commented Jan 7, 2022

the build tags in:
./config/http_config_test.go:// +build go1.8
./config/tls_config_test.go:// +build go1.8
./config/config_test.go:// +build go1.8
./config/http_config.go:// +build go1.8
and the logic in
https://github.com/prometheus/common/blob/main/Makefile#L19 could probably be cleaned up as well

@SuperQ
Copy link
Member Author

SuperQ commented Jan 26, 2022

Ping @LeviHarrison @roidelapluie

mrueg added a commit to mrueg/prometheus-client_golang that referenced this pull request Jan 31, 2022
mrueg added a commit to mrueg/prometheus-client_golang that referenced this pull request Jan 31, 2022
@SuperQ SuperQ requested a review from roidelapluie February 7, 2022 22:11
@SuperQ
Copy link
Member Author

SuperQ commented Feb 7, 2022

Bleh, deprecated use linter errors.

@mrueg
Copy link
Contributor

mrueg commented Feb 22, 2022

@SuperQ I assume those could be treated similar to
"github.com/golang/protobuf/proto" //nolint:staticcheck // Ignore SA1019. Need to keep deprecated package for compatibility.

Let me know if there's anything I can do to get this into a state in which it can be merged.

* Use `make -C dir` instead of `cd dir && make`
* Cleanup uncessary working_directory in config.
* Update Go modules to latest.
* Drop support for old Go versions.
* Cleanup old Go version build tweaks.
* Fixup some deprecated method use.

Signed-off-by: SuperQ <[email protected]>
@SuperQ
Copy link
Member Author

SuperQ commented Feb 28, 2022

Ok, let me know if the changes to expfmt are acceptable.

Copy link
Member

@roidelapluie roidelapluie left a comment

Choose a reason for hiding this comment

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

cc @beorn7 any opinion on the changes in expfmt?

@beorn7
Copy link
Member

beorn7 commented Mar 17, 2022

I think they are fine.

@SuperQ SuperQ merged commit f6b0912 into main Mar 28, 2022
@SuperQ SuperQ deleted the superq/bump_go branch March 28, 2022 08:45
Cori1109 added a commit to Cori1109/client_golang that referenced this pull request Jan 9, 2023
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.

4 participants