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

Fix sorting order of stakes in confirmation time calculations#3159

Merged
solana-grimes merged 1 commit intosolana-labs:masterfrom
pgarg66:beacons_stability
Mar 6, 2019
Merged

Fix sorting order of stakes in confirmation time calculations#3159
solana-grimes merged 1 commit intosolana-labs:masterfrom
pgarg66:beacons_stability

Conversation

@pgarg66
Copy link
Copy Markdown
Contributor

@pgarg66 pgarg66 commented Mar 6, 2019

Problem

The stakes are sorted in incorrect order when confirmation time is calculated.

Summary of Changes

Fixed the sorting order

@pgarg66
Copy link
Copy Markdown
Contributor Author

pgarg66 commented Mar 6, 2019

#3154

@pgarg66 pgarg66 requested review from carllin and sagar-solana March 6, 2019 20:42
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 6, 2019

Codecov Report

Merging #3159 into master will decrease coverage by 0.6%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #3159     +/-   ##
========================================
- Coverage      81%   80.4%   -0.7%     
========================================
  Files         129     129             
  Lines       19370   19515    +145     
========================================
- Hits        15693   15692      -1     
- Misses       3677    3823    +146

@pgarg66 pgarg66 added the automerge Merge this Pull Request automatically once CI passes label Mar 6, 2019
@solana-grimes solana-grimes merged commit 9163e5b into solana-labs:master Mar 6, 2019
@pgarg66 pgarg66 deleted the beacons_stability branch March 6, 2019 21:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants