Skip to content

multiwriter#2

Merged
turkosaurus merged 3 commits into
mainfrom
multiwriter
May 25, 2026
Merged

multiwriter#2
turkosaurus merged 3 commits into
mainfrom
multiwriter

Conversation

@turkosaurus

Copy link
Copy Markdown
Contributor
  • multiwriter
  • add multiwriter support to allow otel

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds additional output options to the logging wrapper by allowing New to write to supplied writers and introducing a fanout logger constructor for additional slog.Handler sinks such as OTEL.

Changes:

  • Adds variadic writer support to New.
  • Adds NewWithHandlers to combine the default tint handler with extra handlers.
  • Adds FanoutHandler implementing slog.Handler fanout behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
log.go Extends logger construction with writer support and handler fanout setup.
fanout.go Adds a new fanout handler implementation for dispatching records to multiple handlers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread fanout.go Outdated
Comment thread fanout.go
Comment thread log.go
Comment thread log.go

@turkosaurus turkosaurus left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

Comment thread fanout.go Outdated
Comment thread log.go Outdated
Comment thread log.go Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@turkosaurus
turkosaurus merged commit 8106888 into main May 25, 2026
3 checks passed
@turkosaurus
turkosaurus deleted the multiwriter branch May 25, 2026 21:35
@turkosaurus
turkosaurus restored the multiwriter branch June 4, 2026 04:48
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