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

Update terminology for block height and genesis block#6782

Merged
jstarry merged 1 commit intosolana-labs:masterfrom
jstarry:one-indexed-block-height
Nov 7, 2019
Merged

Update terminology for block height and genesis block#6782
jstarry merged 1 commit intosolana-labs:masterfrom
jstarry:one-indexed-block-height

Conversation

@jstarry
Copy link
Copy Markdown
Contributor

@jstarry jstarry commented Nov 7, 2019

Problem

Terminology for block height and genesis block is a little off

Summary of Changes

Update terminology and add test for block height

Fixes #6779

@jstarry jstarry requested review from garious and rob-solana November 7, 2019 00:06
@jstarry jstarry force-pushed the one-indexed-block-height branch from a5a1678 to 841d686 Compare November 7, 2019 02:43
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 7, 2019

Codecov Report

Merging #6782 into master will increase coverage by 5.9%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #6782     +/-   ##
========================================
+ Coverage    73.2%   79.2%   +5.9%     
========================================
  Files         219     219             
  Lines       45156   41766   -3390     
========================================
+ Hits        33096   33097      +1     
+ Misses      12060    8669   -3391

@jstarry jstarry changed the title Change bank block height to be one indexed Update terminology for block height and genesis block Nov 7, 2019
@jstarry jstarry merged commit e420800 into solana-labs:master Nov 7, 2019
@jstarry jstarry deleted the one-indexed-block-height branch November 7, 2019 04:09
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.

Bank block height is confusingly zero-indexed

2 participants