Skip to content
Snippets Groups Projects

insights: Add support to sort all series

Merged Warren Gifford requested to merge insights/sort-all-series into main

Created by: CristinaBirkel

Closes https://github.com/sourcegraph/sourcegraph/issues/35711 Builds on https://github.com/sourcegraph/sourcegraph/pull/35873

Description

This moves the sorting step such that all series can be sorted. We've decided that for now it's okay if non-capture group series get sorted by default, given that it just affects the order of the keys.

A note on the tests: I couldn't figure out if it's possible to use some kind of mock resolvers here to mimic what the unit tests were doing before, but if you have any ideas here let me know!

Test plan

Via the UI I verified that capture group insights can still be sorted as expected. And by directly calling the API I verified that we can now also sort a non-capture group series.

Merge request reports

Merged by avatar (May 7, 2025 5:17am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading