Skip to content
This repository has been archived by the owner on Apr 6, 2020. It is now read-only.

Add getLatestHeader() and getLatestBlock() methods, fix saveHeads() bug and update API docs #52

Merged
merged 1 commit into from
May 24, 2018

Conversation

vpulim
Copy link
Contributor

@vpulim vpulim commented May 23, 2018

  • Added new methods to retrieve the latest header and full block
  • Fixed a bug that prevented the latest header and block from being saved to the db
  • Made some API doc changes to help clarify the difference between iterator heads and latest head/block. This hopefully helps explain the difference between the getHead(), getLatestHeader() and getLatestBlock() methods.

@coveralls
Copy link

coveralls commented May 23, 2018

Coverage Status

Coverage increased (+0.08%) to 96.84% when pulling fe0c8e8 on vpulim:fix-head-saving into c4fca13 on ethereumjs:master.

Copy link
Member

@holgerd77 holgerd77 left a comment

Choose a reason for hiding this comment

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

This looks good, thanks for the immediate PR submission, great!

Also tested this locally with my custom state-keeping test file using a leveldown DB, can confirm that this works.

@holgerd77 holgerd77 merged commit 178acbd into ethereumjs:master May 24, 2018
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.

3 participants