Skip to content

CLI: add --sort-by option to list validators by stake/epoch#535

Merged
joncinque merged 6 commits intosolana-program:mainfrom
rishavmehra:feat/sort-by
Jan 28, 2026
Merged

CLI: add --sort-by option to list validators by stake/epoch#535
joncinque merged 6 commits intosolana-program:mainfrom
rishavmehra:feat/sort-by

Conversation

@rishavmehra
Copy link
Contributor

Added --sort-by flag to sort validators by stake and epoch
Useful for identifying top validators or finding validators that need updates

./spl-stake-pool list -v Jito4APyf642JPZPx3hGc6WWJ8zPKtRbRs4P815Awbb --sort-by stake

image

Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Looks good to me overall, just some nits on the naming of params

rishavmehra and others added 2 commits January 27, 2026 15:32
Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Thanks for the quick response!

Sorry, I didn't put all of the places that needed to change in my suggestions, we'll have to make these changes too

rishavmehra and others added 2 commits January 27, 2026 21:05
Co-authored-by: Jon C <me@jonc.dev>
Co-authored-by: Jon C <me@jonc.dev>
@joncinque
Copy link
Contributor

Can you run make format-rust from the top of the repo to fix the formatting issues? After that we should be able to land this

@rishavmehra
Copy link
Contributor Author

Done

Copy link
Contributor

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@joncinque joncinque merged commit 44844c2 into solana-program:main Jan 28, 2026
18 checks passed
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.

2 participants