feat: add new metric helpers for counters and gauges #80
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #80 +/- ##
==========================================
+ Coverage 86.39% 87.52% +1.12%
==========================================
Files 6 6
Lines 522 569 +47
==========================================
+ Hits 451 498 +47
Misses 71 71 ☔ View full report in Codecov by Sentry. |
wsbrunson
force-pushed
the
new-metric-helpers
branch
from
7ca739a
to
25c1346
Compare
|
|
||
| ### `$logger.metricGauge(<event>, <payload>);` | ||
|
|
||
| Queues a gauge metric, helper wrapping `logger.metric` |
There was a problem hiding this comment.
Is gauge a standard term here because I think this could use a bit more detail (I can kind of intuit what it means but it's not obvious)
There was a problem hiding this comment.
it is, that's what its called in the OpenTelemetry library: https://opentelemetry.io/docs/specs/otel/metrics/api/#gauge
|
|
||
| ### Deprecated - `$logger.metric(<event>, <payload>);` | ||
|
|
||
| Queues a metric. We suggest using the `metricCount` or `metricGauge` interface for better type safety and clearer intention in your code. |
No description provided.