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

Remove unused dependencies in ethcore-sync #9589

Merged
merged 3 commits into from
Sep 25, 2018

Conversation

niklasad1
Copy link
Collaborator

Remove unused dependencies and move rustc-hex as test-dependency because it is
only used for tests

Remove unused dependencies and move `rustc-hex` to tests because it is
only used in tests
@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. M5-dependencies 🖇 Dependencies. labels Sep 19, 2018
@5chdn 5chdn added this to the 2.2 milestone Sep 19, 2018
@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Sep 19, 2018
@5chdn 5chdn added A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. and removed A8-looksgood 🦄 Pull request is reviewed well. labels Sep 19, 2018
@5chdn 5chdn added the A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. label Sep 25, 2018
@5chdn
Copy link
Contributor

5chdn commented Sep 25, 2018

Please, resolve conflicts :)

@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodcantmerge 🙄 Pull request is reviewed well, but cannot be merged due to conflicts. A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Sep 25, 2018
@5chdn 5chdn merged commit 3f95a62 into master Sep 25, 2018
@5chdn 5chdn deleted the ethcore-sync/remove-unused-dependencies branch September 25, 2018 13:27
@dvdplm
Copy link
Collaborator

dvdplm commented Sep 25, 2018

Late to the party here, but consider switching to rustc-hex 2.0 or, when possible, use hex-literal (e.g. in tests).

dvdplm added a commit that referenced this pull request Sep 27, 2018
…mon-deps

* origin/master:
  ethereum libfuzzer integration small change (#9547)
  cli: remove reference to --no-ui in --unlock flag help (#9616)
  remove master from releasable branches (#9655)
  ethcore/VerificationQueue don't spawn up extra `worker-threads` when explictly specified not to (#9620)
  RPC: parity_getBlockReceipts (#9527)
  Remove unused dependencies (#9589)
  ignore key_server_cluster randomly failing tests (#9639)
  ethcore: handle vm exception when estimating gas (#9615)
  fix bad-block reporting no reason (#9638)
  Use static call and apparent value transfer for block reward contract code (#9603)
  HF in POA Sokol (2018-09-19) (#9607)
  bump smallvec to 0.6 in ethcore-light, ethstore and whisper (#9588)
  Add constantinople conf to EvmTestClient. (#9570)
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. M5-dependencies 🖇 Dependencies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants