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

Add column for slot range of epoch to epoch-info #7954

Merged
merged 2 commits into from
Jan 24, 2020

Conversation

ryoqun
Copy link
Contributor

@ryoqun ryoqun commented Jan 24, 2020

Another really small PR in continuation of #7838.

I think this will be useful. :)

 ryoqun@ubuqun:~/work/solana/solana$ solana epoch-info
 
 Slot: 128
 Epoch: 2
+Epoch slot range: [96..224)
 Epoch completed percent: 25.000%
 Epoch completed slots: 32/128 (96 remaining)
 Epoch completed time: 12s/51s (38s remaining)

@ryoqun ryoqun requested a review from mvines January 24, 2020 03:57
mvines
mvines previously approved these changes Jan 24, 2020
Copy link
Contributor

@mvines mvines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, useful!

cli/src/cluster_query.rs Outdated Show resolved Hide resolved
Co-Authored-By: Michael Vines <mvines@gmail.com>
@ryoqun ryoqun added the automerge Merge this Pull Request automatically once CI passes label Jan 24, 2020
@ryoqun
Copy link
Contributor Author

ryoqun commented Jan 24, 2020

Thanks for quick review!

@mergify mergify bot dismissed mvines’s stale review January 24, 2020 04:04

Pull request has been modified.

@solana-grimes solana-grimes merged commit 992e985 into solana-labs:master Jan 24, 2020
@codecov
Copy link

codecov bot commented Jan 24, 2020

Codecov Report

Merging #7954 into master will decrease coverage by <.1%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master   #7954     +/-   ##
========================================
- Coverage    81.9%   81.9%   -0.1%     
========================================
  Files         243     243             
  Lines       52551   52556      +5     
========================================
  Hits        43065   43065             
- Misses       9486    9491      +5

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