Skip to content

Conversation

@scttcper
Copy link
Member

Enables empty tag values flag by default. Displays "(empty)" when a specific tag was omitted from some number of events within a group.

Enables empty tag values flag by default. Displays "(empty)" when a specific tag was omitted from some number of events within a group.
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 10, 2025
@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff              @@
##           master   #103097       +/-   ##
============================================
+ Coverage   66.27%    80.63%   +14.35%     
============================================
  Files        9217      9207       -10     
  Lines      393473    393466        -7     
  Branches    25104     24982      -122     
============================================
+ Hits       260792    317254    +56462     
+ Misses     132233     75786    -56447     
+ Partials      448       426       -22     

Comment on lines +211 to +213
has_non_empty_value = any(value != "" for value in result.keys())
if not has_non_empty_value:
raise TagKeyNotFound if group is None else GroupTagKeyNotFound
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the only real change that was breaking some tests. I think they're requesting specific tags and we want to throw if they're not found?

@scttcper scttcper marked this pull request as ready for review November 12, 2025 19:45
@scttcper scttcper requested review from a team as code owners November 12, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants