Skip to content

feat: add safety net for emitting metrics#8

Merged
jvanschie merged 3 commits into
mainfrom
feature/7-swallow-backend-exceptions-on-emit
May 27, 2026
Merged

feat: add safety net for emitting metrics#8
jvanschie merged 3 commits into
mainfrom
feature/7-swallow-backend-exceptions-on-emit

Conversation

@jvanschie

@jvanschie jvanschie commented May 27, 2026

Copy link
Copy Markdown
Contributor

Change Description

Added a safe_emit function to MetricsClient as a safety net to ensure metrics are emitted without exceptions being raised.

Changes Made

  • feat: enhance MetricsClient with safe emit function for safely transmitting metrics via the backend
  • test: add parameterized test for safe emit handling in MetricsClient
  • chore: update test file inclusion pattern in pyproject.toml to support nested directories

Checklist

@jvanschie jvanschie self-assigned this May 27, 2026
@jvanschie
jvanschie requested a review from a team as a code owner May 27, 2026 08:49
@jvanschie
jvanschie force-pushed the feature/7-swallow-backend-exceptions-on-emit branch from 4a91d28 to b5c19e6 Compare May 27, 2026 08:53
@jvanschie jvanschie changed the title feat: add safety net for transmitting metrics feat: add safety net for emitting metrics May 27, 2026
@jvanschie
jvanschie force-pushed the feature/7-swallow-backend-exceptions-on-emit branch 2 times, most recently from 9ce2e74 to 32c4ab1 Compare May 27, 2026 08:58
@jvanschie
jvanschie force-pushed the feature/7-swallow-backend-exceptions-on-emit branch from 4815e9e to 0e4d0ac Compare May 27, 2026 09:27
@jvanschie
jvanschie force-pushed the feature/7-swallow-backend-exceptions-on-emit branch from 0e4d0ac to 34cc13d Compare May 27, 2026 09:28
@sonarqubecloud

Copy link
Copy Markdown

@jvanschie
jvanschie merged commit 3f22f0e into main May 27, 2026
4 checks passed
@jvanschie
jvanschie deleted the feature/7-swallow-backend-exceptions-on-emit branch May 27, 2026 13:04
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.

Error handling toevoegen zodat metrics exceptions niet blocking zijn voor de applicatie

2 participants