Skip to content

fix: impact metric serialization and overload#344

Merged
sjaanus merged 2 commits intomainfrom
imp-fix
Dec 29, 2025
Merged

fix: impact metric serialization and overload#344
sjaanus merged 2 commits intomainfrom
imp-fix

Conversation

@sjaanus
Copy link
Copy Markdown
Contributor

@sjaanus sjaanus commented Dec 29, 2025

Fix enum serialization and add overload.

  1. Previously the enums were going to Unleash in caps and it failed to parse it. It must be small case. Added serialization annotation to solve it.
  2. Noticed we had one missing overload for histogram. Added it.

@github-project-automation github-project-automation Bot moved this from New to Approved PRs in Issues and PRs Dec 29, 2025
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 20570588209

Details

  • 8 of 8 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 79.428%

Totals Coverage Status
Change from base Build 20457872467: 0.2%
Covered Lines: 3210
Relevant Lines: 3893

💛 - Coveralls

@sjaanus sjaanus merged commit 8d75056 into main Dec 29, 2025
9 checks passed
@sjaanus sjaanus deleted the imp-fix branch December 29, 2025 10:31
@github-project-automation github-project-automation Bot moved this from Approved PRs to Done in Issues and PRs Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants