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.

Sign up for GitHub

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

Jump to bottom

Fixed slog.Any does not handle interfaces #27

Merged

david-littlefarmer merged 1 commit into golang-cz:master from noppawitt:master

Dec 19, 2023

Contributor

noppawitt commented Dec 19, 2023

Description

Fixed slog.Any does not handle interfaces addressed in #26

m := map[int]any{0: "a", 1: "b"}
mp := &m

logger.Info("msg",
    slog.Any("m", m),
    slog.Any("mp", mp),
    slog.Any("mpp", &mp),
)

Before:

Now:

Checklist

Code compiles correctly
Created tests which fail without the change (if possible)
All tests passing
Extended the README / documentation, if necessary


          fix slog.Any does not handle interfaces

a2da70c

david-littlefarmer approved these changes

View reviewed changes

Collaborator

david-littlefarmer left a comment

LGTM, thank you for your PR.

david-littlefarmer merged commit 0bb4d22 into golang-cz:master

1 check passed

david-littlefarmer mentioned this pull request

[BUG] - slog.Any does not handle interfaces #26

Closed

renovate bot referenced this pull request in jippi/dottie


          fix(deps): update module github.com/golang-cz/devslog to v0.0.9 in go…

db9b5f2

….mod (#80)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/golang-cz/devslog](https://togithub.com/golang-cz/devslog)
| `v0.0.8` -> `v0.0.9` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fgolang-cz%2fdevslog/v0.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fgolang-cz%2fdevslog/v0.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fgolang-cz%2fdevslog/v0.0.8/v0.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fgolang-cz%2fdevslog/v0.0.8/v0.0.9?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>golang-cz/devslog (github.com/golang-cz/devslog)</summary>

###
[`v0.0.9`](https://togithub.com/golang-cz/devslog/releases/tag/v0.0.9)

[Compare
Source](https://togithub.com/golang-cz/devslog/compare/v0.0.8...v0.0.9)

#### What's Changed

- Fixed slog.Any does not handle interfaces by
[@&#8203;noppawitt](https://togithub.com/noppawitt) in
[https://github.com/golang-cz/devslog/pull/27](https://togithub.com/golang-cz/devslog/pull/27)
- string indent with new lines by
[@&#8203;david-littlefarmer](https://togithub.com/david-littlefarmer) in
[https://github.com/golang-cz/devslog/pull/24](https://togithub.com/golang-cz/devslog/pull/24)
- ignore unexported fields in struct to avoid panic by
[@&#8203;violin0622](https://togithub.com/violin0622) in
[https://github.com/golang-cz/devslog/pull/31](https://togithub.com/golang-cz/devslog/pull/31)
- color tests by
[@&#8203;david-littlefarmer](https://togithub.com/david-littlefarmer) in
[https://github.com/golang-cz/devslog/pull/33](https://togithub.com/golang-cz/devslog/pull/33)
- Stringer by
[@&#8203;david-littlefarmer](https://togithub.com/david-littlefarmer) in
[https://github.com/golang-cz/devslog/pull/32](https://togithub.com/golang-cz/devslog/pull/32)
- Add Stringer support. Fix
[#&#8203;29](https://togithub.com/golang-cz/devslog/issues/29) by
[@&#8203;ghostiam](https://togithub.com/ghostiam) in
[https://github.com/golang-cz/devslog/pull/34](https://togithub.com/golang-cz/devslog/pull/34)
- Removed equal signs from groups by
[@&#8203;david-littlefarmer](https://togithub.com/david-littlefarmer) in
[https://github.com/golang-cz/devslog/pull/36](https://togithub.com/golang-cz/devslog/pull/36)

#### New Contributors

- [@&#8203;noppawitt](https://togithub.com/noppawitt) made their first
contribution in
[https://github.com/golang-cz/devslog/pull/27](https://togithub.com/golang-cz/devslog/pull/27)
- [@&#8203;violin0622](https://togithub.com/violin0622) made their first
contribution in
[https://github.com/golang-cz/devslog/pull/31](https://togithub.com/golang-cz/devslog/pull/31)
- [@&#8203;ghostiam](https://togithub.com/ghostiam) made their first
contribution in
[https://github.com/golang-cz/devslog/pull/34](https://togithub.com/golang-cz/devslog/pull/34)

**Full Changelog**:
golang-cz/devslog@v0.0.8...v0.0.9

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "* */1 * * *" (UTC), 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.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job log](https://developer.mend.io/github/jippi/dottie).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguMjYuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIl19-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet