grafana/pyroscope

util.Logger is NoOpLogger

Aperta

#2435 aperta il 21 set 2023

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)C (528 fork)batch import
good first issue

Metriche repository

Star
 (9041 stelle)
Metriche merge PR
 (Merge medio 5g 7h) (62 PR mergiate in 30 g)

Descrizione

it may hide some logs from us. https://github.com/grafana/pyroscope/pull/2434#discussion_r1332894985 We should rename it or remove

image

### Tasks

Guida contributor

Direzione di ricerca
Individua il tipo `util.Logger` nel codice e analizza la sua implementazione. Controlla tutti gli utilizzi per capire se è intenzionalmente un NoOpLogger o se dovrebbe essere sostituito con un logger reale. Rivedi la discussione nella PR collegata per comprendere l'intento dei manutentori. Decidi se rinominarlo (es. in `NoOpLogger`) o rimuoverlo completamente, e verifica che nessun test dipenda dal nome attuale.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitGo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70