open-feature / go-sdk

Go SDK for OpenFeature
https://openfeature.dev
Apache License 2.0
126 stars 30 forks source link

fix(deps): update module github.com/go-logr/logr to v1.4.0 #241

Closed renovate[bot] closed 8 months ago

renovate[bot] commented 8 months ago

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
github.com/go-logr/logr v1.3.0 -> v1.4.0 age adoption passing confidence

Release Notes

go-logr/logr (github.com/go-logr/logr) ### [`v1.4.0`](https://togithub.com/go-logr/logr/releases/tag/v1.4.0) [Compare Source](https://togithub.com/go-logr/logr/compare/v1.3.0...v1.4.0) This release dramatically improves interoperability with Go's `log/slog` package. In particular, `logr.NewContext` and `logr.NewContextWithSlogLogger` use the same context key, which allows `logr.FromContext` and `logr.FromContextAsSlogLogger` to return `logr.Logger` or `*slog.Logger` respectively, including transparently converting each to the other as needed. Functions `logr/slogr.NewLogr` and `logr/slogr.ToSlogHandler` have been superceded by `logr.FromSlogHandler` and `logr.ToSlogHandler` respectively, and type `logr/slogr.SlogSink` has been superceded by `logr.SlogSink`. All of the old names in `logr/slogr` remain, for compatibility. Package `logr/funcr` now supports `logr.SlogSink`, meaning that it's output passes all but one of the Slog conformance tests (that exception being that `funcr` handles the timestamp itself). Users who have a `logr.Logger` and need a `*slog.Logger` can call `slog.New(logr.ToSlogHandler(...))` and all output will go through the same stack. Users who have a `*slog.Logger` or `slog.Handler` can call `logr.FromSlogHandler(...)` and all output will go through the same stack. ##### What's Changed - slog context support by [@​pohly](https://togithub.com/pohly) in [https://github.com/go-logr/logr/pull/237](https://togithub.com/go-logr/logr/pull/237) - slog support: fix WithGroup + WithValues combination by [@​pohly](https://togithub.com/pohly) in [https://github.com/go-logr/logr/pull/243](https://togithub.com/go-logr/logr/pull/243) - Add tests for context with slog by [@​thockin](https://togithub.com/thockin) in [https://github.com/go-logr/logr/pull/246](https://togithub.com/go-logr/logr/pull/246) - sloghandler: unnamed groups should be inlined by [@​thockin](https://togithub.com/thockin) in [https://github.com/go-logr/logr/pull/245](https://togithub.com/go-logr/logr/pull/245) - Add SlogSink support to funcr by [@​thockin](https://togithub.com/thockin) in [https://github.com/go-logr/logr/pull/241](https://togithub.com/go-logr/logr/pull/241) - funcr: Add LogInfoLevel Option to skip logging level in the info log by [@​spacewander](https://togithub.com/spacewander) in [https://github.com/go-logr/logr/pull/240](https://togithub.com/go-logr/logr/pull/240) ##### New Contributors - [@​spacewander](https://togithub.com/spacewander) made their first contribution in [https://github.com/go-logr/logr/pull/240](https://togithub.com/go-logr/logr/pull/240) **Full Changelog**: https://github.com/go-logr/logr/compare/v1.3.0...v1.4.0

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.



This PR has been generated by Mend Renovate. View repository job log here.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (dbb68e3) 81.61% compared to head (a966d03) 81.61%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #241 +/- ## ======================================= Coverage 81.61% 81.61% ======================================= Files 10 10 Lines 1142 1142 ======================================= Hits 932 932 Misses 192 192 Partials 18 18 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.