-
Notifications
You must be signed in to change notification settings - Fork 784
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.
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
[Merged by Bors] - Export slashing protection per validator #2674
[Merged by Bors] - Export slashing protection per validator #2674
Conversation
Needs some tests and a CLI update before merging |
e3c62f9
to
b33c741
Compare
Done. Now just blocked on #2598 |
b33c741
to
0b177a8
Compare
I'll give this a review once #2598 is merged :) |
0b177a8
to
e44da24
Compare
Rebased on |
Re-running CI as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beautiful!
bors r+ |
## Issue Addressed Part of #2557 ## Proposed Changes Refactor the slashing protection export so that it can export data for a subset of validators. This is the last remaining building block required for supporting the standard validator API (which I'll start to build atop this branch) ## Additional Info Built on and requires #2598
Pull request successfully merged into unstable. Build succeeded: |
Issue Addressed
Part of #2557
Proposed Changes
Refactor the slashing protection export so that it can export data for a subset of validators.
This is the last remaining building block required for supporting the standard validator API (which I'll start to build atop this branch)
Additional Info
Built on and requires #2598