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

Remove unused BlockStatus::Pending #9447

Merged
merged 1 commit into from
Aug 31, 2018
Merged

Conversation

ascjones
Copy link
Contributor

Pending case never instantiated, and only ever matched together with Unknown

Pending case never instantiated, and only ever matched together with Unknown
@ascjones ascjones added A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M4-core ⛓ Core client code / Rust. labels Aug 31, 2018
@ascjones ascjones added A8-looksgood 🦄 Pull request is reviewed well. and removed A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). labels Aug 31, 2018
@ascjones ascjones merged commit bc53e78 into master Aug 31, 2018
@ascjones ascjones deleted the aj/remove-pending-blockstatus branch August 31, 2018 11:13
@5chdn 5chdn added this to the 2.1 milestone Aug 31, 2018
dvdplm added a commit that referenced this pull request Sep 4, 2018
* master:
  evmbin: escape newlines in json errors (#9458)
  use kvdb-* and parity-snappy crates from crates.io (#9441)
  Add EIP-1014 transition config flag (#9268)
  add tags for runner selection of build-linux jobs (#9451)
  Remove unused BlockStatus::Pending (#9447)
  ci: only include local paths in coverage script (except target) (#9437)
  Add POA Networks: Core and Sokol (#9413)
  docker: install missing dependencies in arm target dockerfiles (#9436)
  Random small cleanups (#9423)
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. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants