Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

solana validators now shows current epoch credits instead of lifetime credits #16635

Merged
merged 1 commit into from
Apr 19, 2021

Conversation

mvines
Copy link
Contributor

@mvines mvines commented Apr 19, 2021

Lifetime credits are a fun factoid that nobody really cares about. What you really care about is the credits earned in the current epoch, as this is what influences staking rewards.

@mvines mvines added the v1.6 label Apr 19, 2021
@codecov
Copy link

codecov bot commented Apr 19, 2021

Codecov Report

Merging #16635 (badfca3) into master (015bc03) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

@@            Coverage Diff            @@
##           master   #16635     +/-   ##
=========================================
- Coverage    83.0%    83.0%   -0.1%     
=========================================
  Files         415      415             
  Lines      114705   114705             
=========================================
- Hits        95275    95269      -6     
- Misses      19430    19436      +6     

@brianlong
Copy link
Contributor

@mvines This is a breaking change for me. Can you get this into both v1.5 & v1.6 at the same time? That way I won't need conditional logic in validators.app

@mvines
Copy link
Contributor Author

mvines commented Apr 19, 2021

@mvines This is a breaking change for me. Can you get this into both v1.5 & v1.6 at the same time? That way I won't need conditional logic in validators.app

Oh thanks! Would you prefer that I back this out? Or enable lifetime credits with a flag? Otherwise sure I can backport to v1.5.

Does #16639 cause you any concern?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants