Skip to content
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

Merge validators enhancements #6027

Merged

Conversation

rkapka
Copy link
Contributor

@rkapka rkapka commented May 28, 2020

What type of PR is this?
Other

What does this PR do? Why is it needed?
These PR introduces changes requested in #5968's review comments. These comments were added after the PR had been closed.

Which issues(s) does this PR fix?
N/A

@rkapka rkapka requested a review from a team as a code owner May 28, 2020 19:03
@codecov
Copy link

codecov bot commented May 28, 2020

Codecov Report

Merging #6027 into master will decrease coverage by 0.96%.
The diff coverage is 31.81%.

@@            Coverage Diff             @@
##           master    #6027      +/-   ##
==========================================
- Coverage   60.49%   59.53%   -0.97%     
==========================================
  Files         323      320       -3     
  Lines       28545    27110    -1435     
==========================================
- Hits        17269    16140    -1129     
+ Misses       8977     8776     -201     
+ Partials     2299     2194     -105     

validator/accounts/account.go Show resolved Hide resolved
validator/db/manage.go Outdated Show resolved Hide resolved
validator/db/manage.go Outdated Show resolved Hide resolved
validator/db/manage.go Outdated Show resolved Hide resolved
validator/db/manage.go Outdated Show resolved Hide resolved
@prylabs-bulldozer prylabs-bulldozer bot merged commit d1e2901 into prysmaticlabs:master May 29, 2020
@rkapka rkapka deleted the merge-validators-enhancements branch June 2, 2020 16:26
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.

None yet

3 participants