New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix NPE in `find-stale-pods` #167

Merged

jbrower95 merged 1 commit into master from jb/fix-find-stale-pods

Jan 23, 2025

Contributor

jbrower95 commented Jan 23, 2025

This should've been caught with a test, and we will be adding unit tests over most of the pieces of the CLI this year.
Should be initialized to 0 by default. In the case where all slashed pods had pending withdrawals, this wouldnt have broken, but this is no longer the case.


          fix cli

dfa37ed

jbrower95 requested review from gpsanant and ypatil12

January 23, 2025 01:30

ypatil12 approved these changes

View reviewed changes

jbrower95 merged commit 65a414e into master

3 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet