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

chore!: adopt log/slog, drop go-kit/log #378

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

tjhop
Copy link
Contributor

@tjhop tjhop commented Oct 23, 2024

The bulk of this PR was automated by the following script which is being
used to aid in converting the various exporters/projects to use slog:

https://gist.github.com/tjhop/49f96fb7ebbe55b12deee0b0312d8434

Other changes include:

  • removing unneeded linter settings for go-kit/log
  • enabling sloglint linter

tjhop added 2 commits October 23, 2024 09:35
Also bumps the minimum go version in go.mod to 1.22 to support log/slog,
for exporter-toolkit support.

Signed-off-by: TJ Hoplock <t.hoplock@gmail.com>
The bulk of this PR was automated by the following script which is being
used to aid in converting the various exporters/projects to use slog:

https://gist.github.com/tjhop/49f96fb7ebbe55b12deee0b0312d8434

Other changes include:
- removing unneeded linter settings for go-kit/log
- enabling sloglint linter

Signed-off-by: TJ Hoplock <t.hoplock@gmail.com>
@tjhop
Copy link
Contributor Author

tjhop commented Oct 23, 2024

cc: @SuperQ

@SuperQ SuperQ merged commit 04b47fd into prometheus-community:master Nov 4, 2024
4 checks passed
SuperQ added a commit to kgeckhart/stackdriver_exporter that referenced this pull request Nov 4, 2024
Add prometheus-community#378

Signed-off-by: Ben Kochie <superq@gmail.com>
@SuperQ SuperQ mentioned this pull request Nov 4, 2024
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