Skip to content
This repository was archived by the owner on Feb 3, 2026. It is now read-only.

Comments

feat: Implement the fmt.Stringer interface for User, Group, and Mapping structs#167

Merged
jalseth merged 1 commit intomainfrom
jalseth/logging
Oct 20, 2025
Merged

feat: Implement the fmt.Stringer interface for User, Group, and Mapping structs#167
jalseth merged 1 commit intomainfrom
jalseth/logging

Conversation

@jalseth
Copy link
Contributor

@jalseth jalseth commented Oct 20, 2025

This will make it easier to have more human friendly identifiers in logs.

@jalseth jalseth requested a review from sqin2019 October 20, 2025 19:22
@jalseth jalseth marked this pull request as ready for review October 20, 2025 19:22
@jalseth jalseth requested a review from a team as a code owner October 20, 2025 19:22
@jalseth jalseth requested a review from verbanicm October 20, 2025 19:22
@jalseth jalseth enabled auto-merge (squash) October 20, 2025 19:23
@jalseth jalseth disabled auto-merge October 20, 2025 19:39
@jalseth jalseth changed the title feat: Implement the fmt.Stringer interface for User and Group structs feat: Implement the fmt.Stringer interface for User, Group, and Mapping structs Oct 20, 2025
This will make it easier to have more human friendly identifiers in logs.

Signed-off-by: James Alseth <jalseth@google.com>
@jalseth jalseth merged commit a5b6143 into main Oct 20, 2025
14 checks passed
@jalseth jalseth deleted the jalseth/logging branch October 20, 2025 23:32
jalseth added a commit that referenced this pull request Oct 20, 2025
#167 implemented the Stringer interface
for the User, Group, and Mapping structs. Using this will provie more detailed
info in the logs.

Signed-off-by: James Alseth <jalseth@google.com>
jalseth added a commit that referenced this pull request Oct 21, 2025
…169)

#167 implemented the Stringer
interface for the User, Group, and Mapping structs. Using this will
provie more detailed info in the logs.

Signed-off-by: James Alseth <jalseth@google.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants