-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Conversation
Changes Unknown when pulling d48e262 on rphmeier:pv64 into * on ethcore:master*. |
CLI snapshot height default to 1000 blocks earlier |
@@ -50,7 +50,7 @@ | |||
//! ); | |||
//! let client = Client::new( | |||
//! ClientConfig::default(), | |||
//! ethereum::new_frontier(), | |||
//! ðereum::new_frontier(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this directly related to snapshotting? if not, perhaps a list of any other refactoring/changes made would be helpful, plus comments to guide through the PR would speed along the review process
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it wasn't directly related, although it was useful to assist in adding the SnapshotService
to ClientService
without loading the spec twice. i will refactor it out into another PR.
one minor grumble |
[ci skip]
Closes #1557
The informant is still a little weak, especially on the snapshot creation side of things. This will come in a future PR.