Skip to content

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

Merged
SuperQ merged 2 commits intoprometheus-community:masterfrom
tjhop:chore/adopt-slog
Nov 4, 2024
Merged

chore!: adopt log/slog, drop go-kit/log#378
SuperQ merged 2 commits intoprometheus-community:masterfrom
tjhop:chore/adopt-slog

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
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