grafana/pyroscope

util.Logger is NoOpLogger

Offen

#2.435 geöffnet am 21.09.2023

Auf GitHub ansehen
 (1 Kommentar) (1 Reaktion) (0 zugewiesene Personen)C (528 Forks)batch import
good first issue

Repository-Metriken

Stars
 (9.041 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 5T 7h) (62 gemergte PRs in 30 T)

Beschreibung

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

image

### Tasks

Contributor Guide

Research-Richtung
Finde den Typ `util.Logger` im Code und untersuche seine Implementierung. Überprüfe alle Verwendungen, um festzustellen, ob er absichtlich ein NoOpLogger ist oder ob er durch einen echten Logger ersetzt werden sollte. Sieh dir die Diskussion im verlinkten PR an, um die Absicht der Maintainer zu verstehen. Entscheide, ob du ihn umbenennst (z. B. in `NoOpLogger`) oder komplett entfernst, und stelle sicher, dass keine Tests vom aktuellen Namen abhängen.
Tech Stack
go
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitGo
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70