Skip to content

Comments

Make sort by monitoring structs public#7639

Merged
neilalexander merged 2 commits intonats-io:mainfrom
joeriddles:7638-monitoring-sorting-structs
Dec 15, 2025
Merged

Make sort by monitoring structs public#7639
neilalexander merged 2 commits intonats-io:mainfrom
joeriddles:7638-monitoring-sorting-structs

Conversation

@joeriddles
Copy link
Contributor

closes #7638

Make the structs in monitoring_sort_opts.go public so they can be used in code outside of the server.

Signed-off-by: Joe Riddle joe@synadia.com

@joeriddles joeriddles requested a review from a team as a code owner December 11, 2025 23:27
Signed-off-by: Joe Riddle <joe@synadia.com>
@joeriddles joeriddles force-pushed the 7638-monitoring-sorting-structs branch from b90aed9 to 941b0e1 Compare December 11, 2025 23:28
Signed-off-by: Joe Riddle <joe@synadia.com>
@joeriddles joeriddles force-pushed the 7638-monitoring-sorting-structs branch from f9b43a8 to ac21ca1 Compare December 11, 2025 23:43
Copy link
Member

@MauriceVanVeen MauriceVanVeen left a comment

Choose a reason for hiding this comment

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

LGTM

@neilalexander neilalexander merged commit f68b45b into nats-io:main Dec 15, 2025
44 of 45 checks passed
@joeriddles joeriddles deleted the 7638-monitoring-sorting-structs branch December 15, 2025 14:53
neilalexander added a commit that referenced this pull request Dec 15, 2025
Includes the following:

- #7636
- #7637
- #7643
- #7639
- #7648
- #7634
- #7649

Signed-off-by: Neil Twigg <neil@nats.io>
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.

Expose sorting structs in monitoring_sort_opts.go

3 participants