open-telemetry/opentelemetry-dotnet

Improve instrument name validation and log messages

Offen

#2.532 geöffnet am 27.10.2021

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (889 Forks)auto 404
enhancementhelp wantedkeep-openmetrics

Repository-Metriken

Stars
 (3.725 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T 17h) (112 gemergte PRs in 30 T)

Beschreibung

In PR #2470 we added validation for instrument and view names according to the spec. This issue is to track a small follow up that came up from PR comments :

Contributor Guide

Research-Richtung
Sieh dir die Kommentare zu PR #2470 an, um die Diskussion über Guard Helfer und Logmeldungsverbesserungen zu verstehen; implementiere Guard Helfer für die Namensvalidierung und verbessere die Logmeldung, indem du den ursprünglichen ungültigen Namen angibst.
Tech Stack
csharp
Domain
observability
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.
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
GitC#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70