go-kit/kit

Using prometheus collector MustRegister in New{Counter,Gauge} and possible panic

Open

#937 geöffnet am 14. Nov. 2019

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (2.446 Forks)batch import
bughelp wantednewbie friendly

Repository-Metriken

Stars
 (27.422 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Thanks for the wonderful product! I think it would be fair to point the limitations of calling New in the documentation, since calling New again calls panic.

Contributor Guide

Research-Richtung
Überprüfen Sie die Dokumentation der Funktionen NewCounter und NewGauge und fügen Sie einen Hinweis auf das Risiko einer Panik hinzu, wenn MustRegister zweimal aufgerufen wird.
Tech Stack
go
Domain
backend
Issue Type
Dokumentation
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.
1-3 Stunden
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.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80