Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: metric prefixes #472

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mertalev
Copy link
Contributor

@mertalev mertalev commented Mar 25, 2024

Description

This PR adds the ability to define custom prefixes for metrics. This prefix can be set for custom individual metrics, or for all API metrics. Host metrics are handled through an external dependency, so this is left for future work.

Fixes #265

@pragmaticivan
Copy link
Owner

Hi @mertalev
Thanks for sending this out, the code LGTM, but I might need you to rebase once more, would you mind?

Sorry about the really long delay, but I've been away from OS for a long break and hardly coding typescript in the past few months.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to add Prefix to Metrics
2 participants