Skip to content

TeeLog: Allow arbitrarily many base logs#30

Open
TheBB wants to merge 1 commit intoevalf:mainfrom
TheBB:multitee
Open

TeeLog: Allow arbitrarily many base logs#30
TheBB wants to merge 1 commit intoevalf:mainfrom
TheBB:multitee

Conversation

@TheBB
Copy link
Copy Markdown
Contributor

@TheBB TheBB commented Jun 11, 2020

Should combine nicely with #29 to let you write:

TeeLog(
    FilterLog(DataLog(), minlevel=proto.Level.info),
    FilterLog(StdoutLog(), minlevel=proto.Level.info), maxlevel=proto.Level.user),
    FilterLog(StderrLog(), minlevel=proto.Level.warning),
)

@gertjanvanzwieten
Copy link
Copy Markdown
Contributor

Sorry for the late reply, this fell off the radar. In light of other work we're a bit hesitant at the moment to make changes that we might have difficulty backtracking from if needed, so we prefer to not merge this right away. I'll keep it open though for when that situation clears up.

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