microsoft / microsoft/retina

Remove the Global Logger

Open
#585 0 comments 0 reactions 0 assignees View on GitHub
good first issue help wanted lang/go priority/2 scope/S type/enhancement
Dominant language
Go
Stars
3.2k
Forks
304
Avg merge
1d 16h
Merged PRs (30d)
79

Description

We currently have a global logger in `retinacmd.Logger`. There's never a need for a global logger, and it inhibits the proper use of "child loggers", where certain contextual information is injected into scope-limited logger instances (helping to ensure consistent logging without overhead). We should remove the global logger and inject it in every instance.

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.