9
1682980 - Glinter should lint on metric categories, metric names, labels, reason...
source link: https://bugzilla.mozilla.org/show_bug.cgi?id=1682980
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
Open
Bug 1682980
Opened 3 days ago
Glinter should lint on metric categories, metric names, labels, reasons, and ping names being "too short"
Categories
(Data Platform and Tools :: Glean: SDK, defect, P3)
Data Platform and Tools
▾
Glean: SDK
▾
Tracking
(Not tracked)
People
(Reporter: chutten|PTO, Unassigned)
Details
(Whiteboard: [telemetry:glean-rs:m?])
I was thinking maybe Glinter should complain if your strings are too short.
Here's some WAGs to get us started:
- Metric Category: Min length 2 (I can definitely think of one of length three (
gfx), so 2 might be sensible) - Metric Names: Min length 3 (English has the meaningful word
nowof length 3, so we shouldn't forbid it) - Labels: Min length 2 (Again, I can think of length three (
sun,eOK)) - Reasons: Min length 2 (similar reasons)
- Ping names: Min length 3 (check with ingestion if there's limits on their side)
Thoughts?
You need to log in
before you can comment on or make changes to this bug.
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK