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

engine error: remove faulty/unused From#11404

Merged
ordian merged 1 commit into
masterfrom
na-remove-engine-error-from
Jan 24, 2020
Merged

engine error: remove faulty/unused From#11404
ordian merged 1 commit into
masterfrom
na-remove-engine-error-from

Conversation

@niklasad1
Copy link
Copy Markdown
Collaborator

It is not possible to distinguish which variant to implement with duplicated types such as String, thus these From are just ignored by derive_more which is a hard error in later versions.

Removed some needless format! too

@niklasad1 niklasad1 added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). M4-core ⛓ Core client code / Rust. labels Jan 24, 2020
@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 24, 2020
@ordian ordian merged commit bbc6fb5 into master Jan 24, 2020
@ordian ordian deleted the na-remove-engine-error-from branch January 24, 2020 13:52
dvdplm added a commit that referenced this pull request Jan 29, 2020
…pstream

* master:
  Add POSDAO transition and malice report queue. (#11245)
  update master/nightly version: v2.8.0 (#11419)
  ethcore/res: remove morden testnet (#11392)
  fix: export hardcoded sync format (#11416)
  update hardcoded headers: mainnet and ropsten (#11414)
  AuthorityEngine: Minor cleanups. (#11408)
  Update POA bootnodes (#11411)
  Add EtherCore support (#11402)
  verification: fix race same block + misc (#11400)
  Update ProgPoW to 0.9.3 (#11407)
  update classic testnet bootnodes (#11398)
  dependencies: bump `derive_more v0.99` (#11405)
  engine error: remove faulty/unused `From` (#11404)
  Switching to stable-track (#11377)
  ethcore/res: fix ethereum classic chainspec blake2_f activation block num (#11391)
  Update copyright notice 2020 (#11386)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). 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.

3 participants