-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
R4R: staking spec upgrades / some renaming #3357
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3357 +/- ##
==========================================
+ Coverage 55.51% 55.61% +0.1%
==========================================
Files 141 141
Lines 10445 10437 -8
==========================================
+ Hits 5799 5805 +6
+ Misses 4306 4293 -13
+ Partials 340 339 -1 |
This looks like a fair amount of staking code changes too - just general cleanup? |
@cwgoes yes cleanup - also some name changes per the issue |
Co-Authored-By: rigelrozanski <[email protected]>
Codecov Report
@@ Coverage Diff @@
## develop #3357 +/- ##
===========================================
+ Coverage 61.25% 61.35% +0.09%
===========================================
Files 185 185
Lines 13982 13978 -4
===========================================
+ Hits 8565 8576 +11
+ Misses 4874 4860 -14
+ Partials 543 542 -1 |
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.
Mostly LGTM; a few nits and let's wait for #3507 first.
Co-Authored-By: rigelrozanski <[email protected]>
will wait for #3507 merge before fixes |
This needs a rebase cause #3507 |
closes #3324
There are still slashing sections to be filled in for slashing in
state-transitions.md
but the motivation for me to complete this as a part of this PR doesn't exist, I suggest we merge what is here for now.Targeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added entries in
PENDING.md
with issue #rereviewed
Files changed
in the github PR explorerFor Admin Use: