Skip to content

Conversation

@grypez
Copy link
Contributor

@grypez grypez commented Apr 10, 2025

Refs: #476

  • Adds a stateful Logger class with the ability to pipe logs to multiple endpoints
  • Adds an optional transport constructor argument for specifying endpoints
  • Defines a standardized LoggerEntry type
  • Adds support for logger tags for filtering at the transport level
  • Deprecates makeLogger but keeps it as a wrapper for new Logger

@grypez grypez force-pushed the grypez/log-transport branch from 38eebd0 to f47f01b Compare April 14, 2025 12:29
@grypez grypez force-pushed the grypez/log-transport branch from 8607fde to 7efb416 Compare April 14, 2025 13:07
@grypez grypez force-pushed the grypez/log-transport branch from 7efb416 to d5306ba Compare April 14, 2025 13:23
@grypez grypez marked this pull request as ready for review April 14, 2025 14:02
@grypez grypez requested a review from a team as a code owner April 14, 2025 14:02
@grypez grypez force-pushed the grypez/log-transport branch from 6004f0b to 147f82a Compare April 14, 2025 14:50
Copy link
Contributor

@sirtimid sirtimid left a comment

Choose a reason for hiding this comment

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

LGTM

@grypez grypez merged commit 3c0380b into main Apr 14, 2025
21 checks passed
@grypez grypez deleted the grypez/log-transport branch April 14, 2025 15:59
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.

3 participants