Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Dev/logging configuration #145

Merged
merged 3 commits into from
Apr 28, 2024
Merged

Dev/logging configuration #145

merged 3 commits into from
Apr 28, 2024

Conversation

OG-NI
Copy link
Contributor

@OG-NI OG-NI commented Apr 28, 2024

closes #92

This adds the LoggerService interface for logging functionality which can be used via dependency injection. The usage is documented in logging.md.

@OG-NI OG-NI added the backend Issues related to the backend label Apr 28, 2024
@OG-NI OG-NI requested a review from benedictweis April 28, 2024 18:06
@OG-NI OG-NI self-assigned this Apr 28, 2024
benedictweis
benedictweis approved these changes Apr 28, 2024
View reviewed changes
Copy link
Contributor

@benedictweis benedictweis left a comment

Choose a reason for hiding this comment

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

Looks awesome, thank you very much 👍

@benedictweis benedictweis merged commit 1a2ea1c into main Apr 28, 2024
3 checks passed
@OG-NI OG-NI deleted the dev/logging-configuration branch April 29, 2024 16:03
@OG-NI OG-NI restored the dev/logging-configuration branch April 29, 2024 16:04
@OG-NI OG-NI deleted the dev/logging-configuration branch April 29, 2024 16:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@benedictweis benedictweis benedictweis approved these changes

Assignees

@OG-NI OG-NI

Labels
backend Issues related to the backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Logging-Konfiguration
2 participants
@OG-NI @benedictweis