prometheus / prometheus/client_python
Increase the usage of augmented assignment statements
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 4.4k
- Forks
- 876
- Ø Merge
- 8 T. 4 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
👀 Some source code analysis tools can help to find opportunities for improving software components.
💭 I propose to increase the usage of augmented assignment statements accordingly.
diff --git a/prometheus_client/exposition.py b/prometheus_client/exposition.py
index e374144..1c10055 100644
--- a/prometheus_client/exposition.py
+++ b/prometheus_client/exposition.py
@@ -179,9 +179,9 @@ def generate_latest(registry=REGISTRY):
mtype = metric.type
# Munging from OpenMetrics into Prometheus format.
if mtype == 'counter':
- mname = mname + '_total'
+ mname += '_total'
elif mtype == 'info':
- mname = mname + '_info'
+ mname += '_info'
mtype = 'gauge'
elif mtype == 'stateset':
mtype = 'gauge'
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in prometheus_client/exposition.py bei generate_latest und überprüfe die drei im Issue gezeigten Metriktyp-Zweige. Nimm die gezielten Änderungen an den Augmented-Assignment-Ausdrücken vor und bestätige, dass die generierte Prometheus-Ausgabe unverändert bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- observability
- Issue-Typ
- Refactoring
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 45/100