microsoft/retina

Remove the Global Logger

Open

#585 aberto em 7 de ago. de 2024

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Go (293 forks)auto 404
good first issuehelp wantedlang/gopriority/2scope/Stype/enhancement

Métricas do repositório

Stars
 (3.161 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Pilha de tecnologia
Domain
Tipo Issue
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Prerequisites
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.