Skip to content

Refactor: Metrics - #11

Merged
pboos merged 4 commits into
mainfrom
refactor/metrics
Jun 9, 2023
Merged

Refactor: Metrics#11
pboos merged 4 commits into
mainfrom
refactor/metrics

Conversation

@pboos

@pboos pboos commented Jun 9, 2023

Copy link
Copy Markdown
Contributor
  1. Separate out the MetricsClient
  2. Move metrics logic into MetricsReporter (that now doesn't contain the datadog specific code anymore)
  3. Metric name to not include .error anymore so that it can be used also for .startup.

@pboos
pboos requested review from a team and huguenin June 9, 2023 07:48
@pboos
pboos merged commit 990e921 into main Jun 9, 2023
@pboos
pboos deleted the refactor/metrics branch June 9, 2023 09:05
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.

2 participants