chatmail / chatmail/core

Refactor `LogSink`

Open
#8,555 0 comments 0 reactions 1 assignee Claimed by @j-g00da View on GitHub
Dominant language
Rust
Stars
929
Forks
143
Avg merge
1d 20h
Merged PRs (30d)
53

Description

`EventEmitter` can be moved inside `InnerLogSink`, this will also make spawning a task to pass on the events in `subscribe` redundant. After this, timeouts inside `InnerLogSink::assert` and `InnerLogSink::assert_warns_and_errors` will no longer be needed and can be changed to `try_recv` calls.

Follow up to #8457.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.