Skip to content

multi logger#112

Merged
ruslanti merged 3 commits intomainfrom
feat/multi_logger
Mar 4, 2026
Merged

multi logger#112
ruslanti merged 3 commits intomainfrom
feat/multi_logger

Conversation

@ruslanti
Copy link
Collaborator

@ruslanti ruslanti commented Mar 4, 2026

updates the runtime logging to support multiple log “appenders” per Logger

@ruslanti ruslanti requested a review from Copilot March 4, 2026 07:16
@ruslanti ruslanti self-assigned this Mar 4, 2026
@ruslanti ruslanti added the enhancement New feature or request label Mar 4, 2026
Copy link
Contributor

Copilot AI left a comment

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 updates the runtime logging infrastructure to support multiple log “appenders” per Logger, and updates call sites to use the new construction pattern.

Changes:

  • Refactors runtime::logger::Logger to store multiple appenders and introduces MultiWriter to fan out writes.
  • Updates contexts/tests to construct a logger via Logger::new() and attach appenders via with_appender where needed.
  • Modifies release.toml release configuration (comments out allow-branch).

Reviewed changes

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

File Description
crates/runtime/src/logger.rs Implements multi-appender logger support and a fan-out AsyncWrite (MultiWriter).
src/context.rs Updates logger creation to attach a Console appender under the new API.
crates/http-service/src/executor/http.rs Updates test context logger construction to match the new Logger::new() API.
release.toml Comments out allow-branch, changing release branch restrictions.

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

@ruslanti ruslanti requested review from godronus and qrdl March 4, 2026 07:31
@ruslanti ruslanti merged commit 8ebcac9 into main Mar 4, 2026
5 of 7 checks passed
@ruslanti ruslanti deleted the feat/multi_logger branch March 4, 2026 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants