go-kit/kit

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

Open

#937 aperta il 14 nov 2019

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)Go (2446 fork)batch import
bughelp wantednewbie friendly

Metriche repository

Star
 (27.422 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la documentazione delle funzioni NewCounter e NewGauge e aggiungi una nota sul rischio di panico quando si chiama MustRegister due volte.
Tech stack
go
Dominio
backend
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80