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

introduce ethcore/state module #1953

Merged
merged 3 commits into from
Aug 18, 2016
Merged

introduce ethcore/state module #1953

merged 3 commits into from
Aug 18, 2016

Conversation

rphmeier
Copy link
Contributor

@rphmeier rphmeier commented Aug 18, 2016

mostly just shuffling stuff around and fixing imports

bundles together:

  • state.rs
  • account.rs
  • substate.rs

future work:

  • arbitrary state backends
  • persistent "master state"
  • might put pod_* stuff in there

@rphmeier rphmeier added the A0-pleasereview 🤓 Pull request needs code review. label Aug 18, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.008%) to 86.964% when pulling a5cc40c on state_module into d279794 on master.

@NikVolf
Copy link
Contributor

NikVolf commented Aug 18, 2016

please use move files instead of copy-paste to preserve history and keep diff reasonable

https://githowto.com/moving_files

@NikVolf NikVolf added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. and removed A0-pleasereview 🤓 Pull request needs code review. labels Aug 18, 2016
@rphmeier rphmeier added A0-pleasereview 🤓 Pull request needs code review. and removed A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. labels Aug 18, 2016
@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Aug 18, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.004%) to 87.018% when pulling 8c9f7a3 on state_module into 8159d74 on master.

@NikVolf NikVolf merged commit d7c184b into master Aug 18, 2016
@arkpar arkpar deleted the state_module branch October 3, 2016 15:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants