go-kit/kit

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

Open

#937 ouverte le 14 nov. 2019

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)Go (2 446 forks)batch import
bughelp wantednewbie friendly

Métriques du dépôt

Stars
 (27 422 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez la documentation des fonctions NewCounter et NewGauge et ajoutez une note sur le risque de panique lors de l'appel de MustRegister deux fois.
Stack technique
go
Domaine
backend
Type d'issue
Documentation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80