Skip to content

Conversation

@roaga
Copy link
Member

@roaga roaga commented Oct 14, 2025

Apply the formatting we use for single metric charts to those with group-bys as well. This will fix a bug where charts are not getting processed in Seer correctly

@roaga roaga requested a review from aliu39 October 14, 2025 23:18
@roaga roaga requested a review from a team as a code owner October 14, 2025 23:18
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 14, 2025
cursor[bot]

This comment was marked as outdated.

@roaga roaga enabled auto-merge (squash) October 14, 2025 23:38
@roaga roaga merged commit f0096f5 into master Oct 14, 2025
64 checks passed
@roaga roaga deleted the explorer/fix-grouped-charts branch October 14, 2025 23:47
@codecov
Copy link

codecov bot commented Oct 14, 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   #101493      +/-   ##
===========================================
- Coverage   81.15%    81.05%   -0.11%     
===========================================
  Files        8704      8701       -3     
  Lines      388023    386154    -1869     
  Branches    24400     24400              
===========================================
- Hits       314888    312981    -1907     
- Misses      72784     72822      +38     
  Partials      351       351              

chromy pushed a commit that referenced this pull request Oct 17, 2025
Apply the formatting we use for single metric charts to those with
group-bys as well. This will fix a bug where charts are not getting
processed in Seer correctly
@github-actions github-actions bot locked and limited conversation to collaborators Oct 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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